Skip to content

Conversation

stuartmorgan-g
Copy link
Collaborator

Reverts #9815

The tree is now broken in exactly the way the presubmits for that PR were broken; it should not have landed.

Copy link

@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 reverts a previous change that introduced a MergeSemantics widget to the web Link widget. According to the description, the original change broke the build tree. The changes in this PR correctly remove the feature, its associated integration test, the version bump in pubspec.yaml, and the corresponding CHANGELOG.md entry. The revert appears to be complete and correct.

@stuartmorgan-g stuartmorgan-g added the emergency Override tree-status signal (land even with closed tree), combine with the autosubmit label. label Sep 9, 2025
@flutter-dashboard
Copy link

Detected the emergency label.

If you add the autosubmit label, the bot will wait until all presubmits pass but ignore the tree status, allowing fixes for tree breakages while still validating that they don't break any existing presubmits.

The "Merge" button is also unlocked. To bypass presubmits as well as the tree status, press the GitHub "Add to Merge Queue".

Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@chunhtai chunhtai left a comment

Choose a reason for hiding this comment

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

LGTM

@stuartmorgan-g stuartmorgan-g added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 9, 2025
@auto-submit auto-submit bot merged commit af32689 into main Sep 9, 2025
79 of 80 checks passed
@auto-submit auto-submit bot deleted the revert-9815-fix/link_tab_traversal branch September 9, 2025 17:43
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 10, 2025
github-merge-queue bot pushed a commit to flutter/flutter that referenced this pull request Sep 10, 2025
flutter/packages@2d651b2...03598e7

2025-09-10 [email protected] [url_launcher] Temporarily restore
Android 21 support (flutter/packages#9986)
2025-09-10 [email protected] Roll Flutter from
973320c to a082096 (17 revisions) (flutter/packages#9983)
2025-09-09 [email protected] [various] Scrub pre-Android-API <24
code (flutter/packages#9851)
2025-09-09 [email protected] [google_sign_in] Add a platform
interface method to clear auth tokens (flutter/packages#9929)
2025-09-09 [email protected] Revert "[url_launcher] fix: Link
widget Tab traversal" (flutter/packages#9980)

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-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
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
mboetger pushed a commit to mboetger/flutter that referenced this pull request Sep 18, 2025
…r#175185)

flutter/packages@2d651b2...03598e7

2025-09-10 [email protected] [url_launcher] Temporarily restore
Android 21 support (flutter/packages#9986)
2025-09-10 [email protected] Roll Flutter from
973320c to a082096 (17 revisions) (flutter/packages#9983)
2025-09-09 [email protected] [various] Scrub pre-Android-API <24
code (flutter/packages#9851)
2025-09-09 [email protected] [google_sign_in] Add a platform
interface method to clear auth tokens (flutter/packages#9929)
2025-09-09 [email protected] Revert "[url_launcher] fix: Link
widget Tab traversal" (flutter/packages#9980)

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-flutter-autoroll
Please CC [email protected] on the revert to ensure that a
human
is aware of the problem.

To file a bug in Flutter:
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
autosubmit Merge PR when tree becomes green via auto submit App emergency Override tree-status signal (land even with closed tree), combine with the autosubmit label. p: url_launcher platform-web
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants