Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "[webview_flutter] Improve flaky scroll tests" #7652

Merged
merged 1 commit into from
Sep 17, 2024

Conversation

stuartmorgan
Copy link
Contributor

Reverts #7621

The re-enabled tests are failing in post-submit.

Alternately, instead of reverting we could just re-add the skips with the link, if we still want to keep the updated version as a better foundation to work from.

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. I'm fine with just reverting.

@tarrinneal tarrinneal added the warning: land on red to fix tree breakage Override tree-status signal (land even with closed tree), combine with the autosubmit label. label Sep 17, 2024
@tarrinneal
Copy link
Contributor

Landing on red to fix the tree?

@tarrinneal tarrinneal merged commit d7e6e20 into main Sep 17, 2024
75 of 76 checks passed
@tarrinneal tarrinneal deleted the revert-7621-improve_scroll_test branch September 17, 2024 00:13
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 17, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 17, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 17, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Sep 17, 2024
flutter/packages@df88c81...4f2b9cd

2024-09-17 engine-flutter-autoroll@skia.org Roll Flutter from 5d83a98 to c4c9f47 (15 revisions) (flutter/packages#7656)
2024-09-17 engine-flutter-autoroll@skia.org Roll Flutter from 2d30fe4 to 5d83a98 (5 revisions) (flutter/packages#7650)
2024-09-17 stuartmorgan@google.com Revert "[webview_flutter] Improve flaky scroll tests" (flutter/packages#7652)

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 flutter-ecosystem@google.com,rmistry@google.com 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
p: webview_flutter platform-android platform-ios warning: land on red to fix tree breakage Override tree-status signal (land even with closed tree), combine with the autosubmit label.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants