Skip to content

Remove use of deprecated API related to removal of the VM isolate.#187013

Merged
auto-submit[bot] merged 1 commit into
flutter:masterfrom
rmacnak-google:narrow-remove-vm-isolate
Jun 1, 2026
Merged

Remove use of deprecated API related to removal of the VM isolate.#187013
auto-submit[bot] merged 1 commit into
flutter:masterfrom
rmacnak-google:narrow-remove-vm-isolate

Conversation

@rmacnak-google
Copy link
Copy Markdown
Contributor

@rmacnak-google rmacnak-google commented May 23, 2026

  • Don't pass VM snapshot to Dart_Initialize.
  • Don't expect kDart{Vm,Isolate}Snapshot{Data,Instructions} symbols defined.
  • Replace 4-symbol Dart_LoadELF with 2-symbol Dart_LoadELF2.

Blocking: https://dart-review.googlesource.com/c/sdk/+/502780

@flutter-dashboard flutter-dashboard Bot added the CICD Run CI/CD label May 23, 2026
@github-actions github-actions Bot added tool Affects the "flutter" command-line tool. See also t: labels. engine flutter/engine related. See also e: labels. platform-fuchsia Fuchsia code specifically labels May 23, 2026
@rmacnak-google rmacnak-google force-pushed the narrow-remove-vm-isolate branch from 45c0dea to cb5002d Compare May 23, 2026 20:50
@github-actions github-actions Bot removed the CICD Run CI/CD label May 23, 2026
@rmacnak-google rmacnak-google added the CICD Run CI/CD label May 23, 2026
@rmacnak-google rmacnak-google force-pushed the narrow-remove-vm-isolate branch from cb5002d to 17c646e Compare May 23, 2026 21:25
@github-actions github-actions Bot removed the CICD Run CI/CD label May 23, 2026
@rmacnak-google rmacnak-google added the CICD Run CI/CD label May 23, 2026
@rmacnak-google rmacnak-google force-pushed the narrow-remove-vm-isolate branch from 17c646e to ad29d1a Compare May 23, 2026 22:15
@github-actions github-actions Bot removed tool Affects the "flutter" command-line tool. See also t: labels. CICD Run CI/CD labels May 23, 2026
@rmacnak-google rmacnak-google added the CICD Run CI/CD label May 23, 2026
@rmacnak-google rmacnak-google marked this pull request as ready for review May 26, 2026 17:34
@rmacnak-google
Copy link
Copy Markdown
Contributor Author

This is a narrower change just to removed the deprecated APIs. I have a bigger clean up change in progress: #186407.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request refactors Dart VM and isolate snapshot handling by consolidating snapshot symbols and replacing the separate VM and isolate snapshot data/instructions with unified snapshot symbols (such as kDartSnapshotData and kDartSnapshotText). It updates the GN build files, C++ runtime sources, Fuchsia runner configurations, and Devicelab tests to align with these changes, including replacing Dart_LoadELF with Dart_LoadELF2. There are no review comments to address, and I have no additional feedback to provide.

@rmacnak-google rmacnak-google added the autosubmit Merge PR when tree becomes green via auto submit App label May 26, 2026
@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 26, 2026
@auto-submit
Copy link
Copy Markdown
Contributor

auto-submit Bot commented May 26, 2026

autosubmit label was removed for flutter/flutter/187013, because - The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

@rmacnak-google rmacnak-google added autosubmit Merge PR when tree becomes green via auto submit App CICD Run CI/CD and removed CICD Run CI/CD labels May 26, 2026
@fluttergithubbot
Copy link
Copy Markdown
Contributor

An existing Git SHA, ad29d1ad937ae1597db265d9fd15e2f3de28bfa5, was detected, and no actions were taken.

To re-trigger presubmits after closing or re-opeing a PR, or pushing a HEAD commit (i.e. with --force) that already was pushed before, push a blank commit (git commit --allow-empty -m "Trigger Build") or rebase to continue.

@rmacnak-google rmacnak-google force-pushed the narrow-remove-vm-isolate branch from ad29d1a to 6da5400 Compare May 27, 2026 17:26
@github-actions github-actions Bot removed the CICD Run CI/CD label May 27, 2026
@rmacnak-google rmacnak-google added the CICD Run CI/CD label May 27, 2026
@auto-submit
Copy link
Copy Markdown
Contributor

auto-submit Bot commented May 27, 2026

autosubmit label was removed for flutter/flutter/187013, because - The status or check suite Linux linux_host_engine_ddm has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submit auto-submit Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label May 27, 2026
@rmacnak-google rmacnak-google added the autosubmit Merge PR when tree becomes green via auto submit App label May 31, 2026
@auto-submit auto-submit Bot added this pull request to the merge queue Jun 1, 2026
Merged via the queue into flutter:master with commit 0d806c9 Jun 1, 2026
207 checks passed
@flutter-dashboard flutter-dashboard Bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jun 1, 2026
auto-submit Bot pushed a commit to flutter/packages that referenced this pull request Jun 1, 2026
flutter/flutter@b05a9d7...54e199a

2026-06-01 engine-flutter-autoroll@skia.org Roll Fuchsia Linux SDK from jMR_VXQi07kAk8vbR... to q27k7_um1GvVrySZS... (flutter/flutter#187338)
2026-06-01 rmacnak@google.com Remove use of deprecated API related to removal of the VM isolate. (flutter/flutter#187013)
2026-06-01 116356835+AbdeMohlbi@users.noreply.github.com Improve `dependOnInheritedWidgetOfExactType` documentation to explain why it is bad to use it in initState (flutter/flutter#186216)
2026-06-01 chris@bracken.jp Revert "Move dart-lang/ai to a top level third party dependency in en… (flutter/flutter#187370)
2026-05-30 jakemac@google.com Move dart-lang/ai to a top level third party dependency in engine (flutter/flutter#187268)
2026-05-30 evanwall@buffalo.edu add sdf golden variants for OpenGL (flutter/flutter#187246)
2026-05-30 engine-flutter-autoroll@skia.org Roll Skia from dc01525ac468 to 0aee4675e0ad (6 revisions) (flutter/flutter#187334)
2026-05-29 engine-flutter-autoroll@skia.org Roll Skia from c480ba2eb2eb to dc01525ac468 (4 revisions) (flutter/flutter#187317)
2026-05-29 jason-simmons@users.noreply.github.com Remove the Y coordinate flip workaround in the Material stretch effect shader now that it is no longer required by the Impeller GLES back end (flutter/flutter#187247)
2026-05-29 bkonyi@google.com [flutter_tools, devicelab] Fix filesystem safety guard for symlinked temp directories (flutter/flutter#187320)
2026-05-29 30870216+gaaclarke@users.noreply.github.com Brings linux tests out of bringup. (flutter/flutter#187271)
2026-05-29 137456488+flutter-pub-roller-bot@users.noreply.github.com Roll pub packages (flutter/flutter#187321)
2026-05-29 engine-flutter-autoroll@skia.org Roll Fuchsia GN SDK from SEfYx3xgueX3aFAY3... to oOAcFhkoE2_-Sy67z... (flutter/flutter#187310)
2026-05-29 36861262+QuncCccccc@users.noreply.github.com Fix mismatch between hit-test tree and traversal tree (flutter/flutter#186826)
2026-05-29 jason-simmons@users.noreply.github.com [Impeller] Ensure that the TextureGLES destructor cleans up all objects that it holds including the sync fence (flutter/flutter#187216)
2026-05-29 engine-flutter-autoroll@skia.org Roll Packages from 10cbdc5 to e930ced (3 revisions) (flutter/flutter#187306)
2026-05-29 engine-flutter-autoroll@skia.org Roll Skia from d9d6b440c4e7 to c480ba2eb2eb (1 revision) (flutter/flutter#187305)

If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages
Please CC stuartmorgan@google.com,tarrinneal@google.com on the revert to ensure that a human
is aware of the problem.

To file a bug in Packages: https://github.com/flutter/flutter/issues/new/choose

To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CICD Run CI/CD engine flutter/engine related. See also e: labels. platform-fuchsia Fuchsia code specifically

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants