[CP] Update with DevTools that does not fail heap snapshotting. about sdk

[jacob314]

LGTM

[sortie]

Can you please add more detail to your answers? I'm not one of the people that approve cherry-pick requests, but there is almost no context here. I designed the current form of the cherry-pick process and our style is to write the cherry-pick request so people without context can understand what the issue was, what the fix is, and why it is safe. You should think of this issue as the entry point for people that want an overview of what happened and why. @itsjustkevin

Issue Description
#55475

Please summarize the issue. The reason we have this separate issue is because the original issues are usually an investigation and it's much more work to read the fully issue and locate the conclusions :)

What is the fix
Upgraded version of vm_service in DevTools to 14.2.1

What changed in the vm_service code to fix the problem?

Why cherry-pick
DevTools sometimes fails to snapshot memory with vm_service to 14.2.0

OK that sounds bad.

@kenzieschmoll raises doubts #55475 (comment) that the cherry-pick might not be needed after all? Can you confirm whether a cherry-pick is needed?

No visible risk

Can you please elaborate in more detail about why this cherry-pick is safe? Changes always have the potential for new bugs. If a mistake was made, it could result in a broken 3.4.0 release when the beta period is over, so we usually want to know the change is safe. E.g. is there any tests that catch this problem and confirm it no longer exists? For instance, did vm_service 14.2.1 get tested with devtools?

[polina-c]

Thank you!
Fixed description and replied #55475 (comment)

[sortie]

#55475 (comment) points out the devtools CP build somehow still contains vm_service 14.2.0 and a new CP build would be required.

[polina-c]

#55475 (comment) points out the devtools CP build somehow still contains vm_service 14.2.0 and a new CP build would be required.

Nope, it does not. That comment was made before I created CP to point out to the problem of the existing version of DevTools for beta, explaining why we need CP. There are two CPs in this issue: one for Dart SDK and one for DevTools.

[vsmenon]

lgtm