Skip to content

Conversation

@auto-submit
Copy link
Contributor

@auto-submit auto-submit bot commented Jan 24, 2024

Reverts #142062
Initiated by: eliasyishak
This change reverts the following previous change:
Original Description:
This PR makes no behavioral changes to executed code, and instead focuses on organization and naming:

  1. Almost1 anything named external_ui is renamed external_textures
  2. Extended the README to explain the intent of the test, as well as how to run it
  3. Renamed main.dart and main_test.dart to frame_rate_main.dart and frame_rate_test.dart (we'll add more)
  4. Did some refactoring of the test to make it more obvious what is being asserted (i.e. widgetBuilds and friends)

Given how complex (and in-flux) this directory is, I'm also requesting either John, Jonah or I review any changes.

Footnotes

  1. Except the name of the .ci.yaml task, i.e. name: Linux_pixel_7pro external_ui_integration_test because I'm apparently not able to change that without creating a new task as bringup: true and playing a bit of a dance. Maybe that's worth doing though (in future PRs)?

@auto-submit auto-submit bot added the revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests. label Jan 24, 2024
@auto-submit auto-submit bot merged commit 54e9f2d into master Jan 24, 2024
@auto-submit auto-submit bot deleted the revert_2e2042ff8e4b19109ecba2d78712e9381c46900d branch January 24, 2024 21:41
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Jan 25, 2024
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Jan 25, 2024
Roll Flutter from 19b06f4 to a8efa77 (38 revisions)

flutter/flutter@19b06f4...a8efa77

2024-01-25 [email protected] Roll Flutter Engine from e2014f007f61 to 7c4ed15cb271 (1 revision) (flutter/flutter#142221)
2024-01-25 [email protected] Roll Flutter Engine from 5fa2e2920274 to e2014f007f61 (1 revision) (flutter/flutter#142213)
2024-01-25 [email protected] provide command to `FakeCommand::onRun` (flutter/flutter#142206)
2024-01-25 [email protected] Roll Flutter Engine from c346fd3d9ca1 to 5fa2e2920274 (1 revision) (flutter/flutter#142212)
2024-01-25 [email protected] Roll Flutter Engine from d4d8f668159b to c346fd3d9ca1 (1 revision) (flutter/flutter#142209)
2024-01-25 [email protected] Roll Flutter Engine from 499ed00bbda2 to d4d8f668159b (2 revisions) (flutter/flutter#142205)
2024-01-25 [email protected] Roll Flutter Engine from 4880592ca5ba to 499ed00bbda2 (6 revisions) (flutter/flutter#142202)
2024-01-25 [email protected] [ci] Adds test for web hot restart with const App. (flutter/flutter#141824)
2024-01-25 [email protected] Migrate android_view to linux_android_emu platform. (flutter/flutter#142184)
2024-01-25 [email protected] Refactor `external_ui` without making any name changes (I think) (flutter/flutter#142192)
2024-01-25 [email protected] Fix text selection edge scrolling when inside a horizontal scrollable (flutter/flutter#140250)
2024-01-25 [email protected] Roll Flutter Engine from d7bf5ec1dcdd to 4880592ca5ba (2 revisions) (flutter/flutter#142186)
2024-01-24 [email protected] Roll Flutter Engine from 6a7f963dc751 to d7bf5ec1dcdd (2 revisions) (flutter/flutter#142185)
2024-01-24 [email protected] Reland "Remove hack from PageView." (flutter/flutter#142172)
2024-01-24 [email protected] Upgrade leak_tracker. (flutter/flutter#142162)
2024-01-24 [email protected] Migrate android views to devicelab. (flutter/flutter#142081)
2024-01-24 [email protected] Roll Flutter Engine from ed498f111d53 to 6a7f963dc751 (4 revisions) (flutter/flutter#142176)
2024-01-24 [email protected] Support wireless debugging for iOS 12 or earlier (flutter/flutter#141439)
2024-01-24 [email protected] Roll Flutter Engine from 4bc368ee5f74 to ed498f111d53 (5 revisions) (flutter/flutter#142167)
2024-01-24 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Refactor `external_ui` � `external_textures`" (flutter/flutter#142173)
2024-01-24 [email protected] Refactor `external_ui` â�� `external_textures` (flutter/flutter#142062)
2024-01-24 [email protected] Roll Flutter Engine from 0b9fce355df9 to 4bc368ee5f74 (3 revisions) (flutter/flutter#142157)
2024-01-24 [email protected] Update navigationBar label's maxScaleFactor to meet GAR requirement (flutter/flutter#141998)
2024-01-24 [email protected] Roll Flutter Engine from b65556fa543e to 0b9fce355df9 (1 revision) (flutter/flutter#142147)
2024-01-24 [email protected] Marks Mac_arm64 build_tests_2_4 to be unflaky (flutter/flutter#142115)
2024-01-24 [email protected] Marks Mac_x64 build_tests_2_4 to be unflaky (flutter/flutter#142111)
2024-01-24 [email protected] Marks Mac_x64 build_tests_1_4 to be unflaky (flutter/flutter#142110)
2024-01-24 [email protected] Marks Mac_x64 build_tests_3_4 to be unflaky (flutter/flutter#142112)
2024-01-24 [email protected] Revise tooltip theme docs, including more cross-references (flutter/flutter#137316)
2024-01-24 [email protected] Run some tests explicitly in both arm and x64. (flutter/flutter#141910)
2024-01-24 [email protected] Roll Flutter Engine from ba3a70ce7722 to b65556fa543e (3 revisions) (flutter/flutter#142140)
2024-01-24 [email protected] Fixes #138773, port autocomplete to OverlayPortal (flutter/flutter#140285)
2024-01-24 [email protected] Revert "[web] - Fix broken `TextField` in semantics mode when it's a sibling of `Navigator`" (flutter/flutter#142129)
2024-01-24 [email protected] Marks Mac_arm64 build_tests_1_4 to be unflaky (flutter/flutter#142114)
2024-01-24 [email protected] Roll Packages from 841fe90 to 8fbdf65 (2 revisions) (flutter/flutter#142139)
2024-01-24 [email protected] Marks Mac_arm64 build_tests_3_4 to be unflaky (flutter/flutter#142116)
2024-01-24 [email protected] Marks Mac_x64 build_tests_4_4 to be unflaky (flutter/flutter#142113)
2024-01-24 [email protected] Marks Mac_arm64 build_tests_4_4 to be unflaky (flutter/flutter#142117)

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 [email protected],[email protected],[email protected] 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
...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

revert of Bot Only: Tracking label for bot. Tracks new revert of pull requests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants