-
Notifications
You must be signed in to change notification settings - Fork 6k
Move scenario app to use iPhone Se 3rd gen iOS 16.2, un-skip test TwoPlatformViewsWithOtherBackDropFilterTests
#41532
Conversation
update the test simulator update proj file
27a9b37
to
3c5b966
Compare
TwoPlatformViewsWithOtherBackDropFilterTests
Well, my local arm failed the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think git is confused and compared the wrong file. You can check the full file names by hovering the mouse to file name above. Same as the one below.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can land this now, rebase #41640 onto it, and either update ios-16-0_14a5294e
to ios-16-2_14c18
or remove totally and update the images again.
https://github.com/flutter/engine/pull/41640/files#r1181847615
…p test `TwoPlatformViewsWithOtherBackDropFilterTests` (flutter/engine#41532)
…p test `TwoPlatformViewsWithOtherBackDropFilterTests` (flutter/engine#41532)
…p test `TwoPlatformViewsWithOtherBackDropFilterTests` (flutter/engine#41532)
…125845) flutter/engine@3fa6084...7d87410 2023-05-01 ychris@google.com Update xcode to 14e222b (flutter/engine#41640) 2023-05-01 aam@google.com Run flutter license script with dart `--intrepret_irregexp` runtime option (flutter/engine#41646) 2023-05-01 skia-flutter-autoroll@skia.org Roll Skia from 82d1ef7a833e to 38e56b6da8f9 (8 revisions) (flutter/engine#41645) 2023-05-01 ychris@google.com Move scenario app to use iPhone Se 3rd gen iOS 16.2, un-skip test `TwoPlatformViewsWithOtherBackDropFilterTests` (flutter/engine#41532) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Please CC jsimmons@google.com,rmistry@google.com,zra@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://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
iPhone 8 is old and not installed by default with the latest XCodes. Move the scenario test to a newer iPhone that has a home button. (Unfortunately, iPhones without home button have the bottom navigation bar that animates, resulting inconsistency during screenshots)
Fixes flutter/flutter#125167
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.