Skip to content

Revert "[Windows] Enable merged platform and UI thread by default" #167420

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

Merged

Conversation

loic-sharma
Copy link
Member

This reverts commit afdc861.

See: #167418

@loic-sharma loic-sharma requested review from knopp and matanlurey April 18, 2025 21:10
@github-actions github-actions bot added engine flutter/engine repository. See also e: labels. platform-windows Building on or for Windows specifically a: desktop Running on desktop labels Apr 18, 2025
@matanlurey
Copy link
Contributor

If there is a fix-forward I am OK with that, given the 1-2 tests failing seem perhaps trivial?

Up to you though, will LGTM optimistically.

@loic-sharma
Copy link
Member Author

loic-sharma commented Apr 18, 2025

If there is a fix-forward I am OK with that, given the 1-2 tests failing seem perhaps trivial?

I'd like to use this PR to unblock the tree instead of #167419 so that we can keep Windows tests as required

@loic-sharma loic-sharma added the autosubmit Merge PR when tree becomes green via auto submit App label Apr 18, 2025
@auto-submit auto-submit bot added this pull request to the merge queue Apr 18, 2025
Merged via the queue into flutter:master with commit 66ecc88 Apr 18, 2025
179 checks passed
@flutter-dashboard flutter-dashboard bot removed the autosubmit Merge PR when tree becomes green via auto submit App label Apr 18, 2025
github-merge-queue bot pushed a commit that referenced this pull request Apr 18, 2025
<!-- start_original_pr_link -->
Reverts: #167419
<!-- end_original_pr_link -->
<!-- start_initiating_author -->
Initiated by: loic-sharma
<!-- end_initiating_author -->
<!-- start_revert_reason -->
Reason for reverting: Test should no longer be flaky now that
#167420 has landed.
<!-- end_revert_reason -->
<!-- start_original_pr_author -->
Original PR Author: matanlurey
<!-- end_original_pr_author -->

<!-- start_reviewers -->
Reviewed By: {jtmcdole}
<!-- end_reviewers -->

<!-- start_revert_body -->
This change reverts the following previous change:
See #167418.
<!-- end_revert_body -->

Co-authored-by: auto-submit[bot] <[email protected]>
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 19, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 19, 2025
engine-flutter-autoroll added a commit to engine-flutter-autoroll/packages that referenced this pull request Apr 20, 2025
auto-submit bot pushed a commit to flutter/packages that referenced this pull request Apr 20, 2025
flutter/flutter@3ed38e2...cfb887c

2025-04-20 [email protected] Roll Fuchsia Linux SDK from 0U_vEALFF7qRJZ_cE... to RGmU4KeQhrxqE7hsr... (flutter/flutter#167447)
2025-04-19 [email protected] Roll Skia from 3dc3ffeb45f0 to bd9ef4955aad (1 revision) (flutter/flutter#167434)
2025-04-19 [email protected] Roll Fuchsia Linux SDK from MwYckh5OvwwmIYLx0... to 0U_vEALFF7qRJZ_cE... (flutter/flutter#167430)
2025-04-19 [email protected] Roll Skia from 177a2929e32f to 3dc3ffeb45f0 (1 revision) (flutter/flutter#167428)
2025-04-19 [email protected] Revert "Reduce app startup latency by initializing the engine on a separate thread (#166918)" (flutter/flutter#167427)
2025-04-18 [email protected] Fix keyboard cover SearchAnchor list results (flutter/flutter#165382)
2025-04-18 98614782+auto-submit[bot]@users.noreply.github.com Reverts "Mark `windows_host_engine_test` flaky (#167419)" (flutter/flutter#167425)
2025-04-18 [email protected] Docs: Update date picker theme day color properties doc (flutter/flutter#166122)
2025-04-18 [email protected] Add ktlint test for generated files from templates (flutter/flutter#167378)
2025-04-18 [email protected] Revert "[Windows] Enable merged platform and UI thread by default" (flutter/flutter#167420)
2025-04-18 [email protected] Add a DrivenScrollActivity.simulation constructor (flutter/flutter#166730)
2025-04-18 [email protected] Added missing period for consistency and readability (flutter/flutter#162887)
2025-04-18 [email protected] Roll Skia from 6c4595124690 to 177a2929e32f (1 revision) (flutter/flutter#167417)
2025-04-18 [email protected] Fix codesigning for `Mac_arm64_ios imitation_game_flutter` (flutter/flutter#167307)
2025-04-18 [email protected] Mark `windows_host_engine_test` flaky (flutter/flutter#167419)
2025-04-18 [email protected] Roll Dart SDK from ab60afc99bcb to 0cfefe2a03fb (1 revision) (flutter/flutter#167414)
2025-04-18 [email protected] [web] close input connection when window/iframe loses focus (flutter/flutter#166804)

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] 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
knopp added a commit to knopp/flutter that referenced this pull request Apr 21, 2025
jason-simmons added a commit to jason-simmons/flutter that referenced this pull request Apr 22, 2025
knopp added a commit to knopp/flutter that referenced this pull request Apr 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: desktop Running on desktop engine flutter/engine repository. See also e: labels. platform-windows Building on or for Windows specifically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants