Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

[engine] make UI thread the platform thread for Android. Still allows opt out as g3 escape hatch. #55111

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

jonahwilliams
Copy link
Member

@@ -231,7 +231,7 @@ struct Settings {

// If true, the UI thread is the platform thread on supported
// platforms.
bool merged_platform_ui_thread = false;
bool merged_platform_ui_thread = true;
Copy link
Member Author

Choose a reason for hiding this comment

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

Note: this is only used by android now. Can be removed in a few weeks.

@@ -208,12 +208,6 @@ static BOOL DoesHardwareSupportWideGamut() {
settings.enable_dart_profiling = enableDartProfiling.boolValue;
}

NSNumber* enableMergedPlatformUIThread =
Copy link
Member Author

Choose a reason for hiding this comment

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

this is unused.

Copy link
Member

@jtmcdole jtmcdole left a comment

Choose a reason for hiding this comment

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

LDGTM

@jonahwilliams jonahwilliams added the autosubmit Merge PR when tree becomes green via auto submit App label Sep 12, 2024
@auto-submit auto-submit bot merged commit dcfecdb into flutter:main Sep 12, 2024
32 checks passed
@jonahwilliams jonahwilliams deleted the merge_threads branch September 12, 2024 16:38
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 12, 2024
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Sep 12, 2024
auto-submit bot pushed a commit to flutter/flutter that referenced this pull request Sep 12, 2024
…155128)

flutter/engine@48ddaf5...8609af6

2024-09-12 [email protected] Revert "Update Color to do all calculations with floating point components" (flutter/engine#55153)
2024-09-12 [email protected] Migrate more tests from `litetest` to `package:test` (flutter/engine#55119)
2024-09-12 [email protected] Roll Skia from acff7f24ddbe to 26b048c6a53b (1 revision) (flutter/engine#55151)
2024-09-12 [email protected] Roll Dart SDK from a438066d634f to aa27c61f5859 (8 revisions) (flutter/engine#55147)
2024-09-12 [email protected] Update Skia build for Vulkan headers (flutter/engine#55143)
2024-09-12 [email protected] Roll Skia from 2b40b50ea423 to acff7f24ddbe (1 revision) (flutter/engine#55144)
2024-09-12 [email protected] Update Color to do all calculations with floating point components (flutter/engine#54981)
2024-09-12 [email protected] Roll Fuchsia Linux SDK from fKNT8lbGh8JzxjE6m... to 3YH1DEYJ-s93fHBw5... (flutter/engine#55142)
2024-09-12 [email protected] Migrate `const_finder_test` to use `package:test` (flutter/engine#55132)
2024-09-12 [email protected] [engine] make UI thread the platform thread for Android. Still allows opt out as g3 escape hatch. (flutter/engine#55111)
2024-09-12 [email protected] Roll Skia from b750cbedc114 to 2b40b50ea423 (1 revision) (flutter/engine#55141)

Also rolling transitive DEPS:
  fuchsia/sdk/core/linux-amd64 from fKNT8lbGh8Jz to 3YH1DEYJ-s93

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 [email protected],[email protected],[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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
autosubmit Merge PR when tree becomes green via auto submit App platform-android platform-ios
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants