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

Always make gpu thread different from platform thread regardless of platform view #16068

Merged
merged 4 commits into from
Jan 28, 2020

Conversation

cyanglaz
Copy link
Contributor

@cyanglaz cyanglaz commented Jan 27, 2020

This PR removes the conditional configuration that makes GPU thread the same as platform thread when platform view is enabled.
This was blocked by flutter/flutter#38903, and the issue is resolved, we are able to separate the threads now

Fixes flutter/flutter#23975

Copy link
Contributor

@amirh amirh left a comment

Choose a reason for hiding this comment

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

LGTM

@cyanglaz cyanglaz merged commit 0889e97 into flutter:master Jan 28, 2020
cyanglaz pushed a commit that referenced this pull request Jan 28, 2020
cyanglaz pushed a commit that referenced this pull request Jan 28, 2020
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request Jan 29, 2020
engine-flutter-autoroll added a commit to flutter/flutter that referenced this pull request Jan 29, 2020
flutter/engine@c4229bf...ec32966

git log c4229bf..ec32966 --first-parent --oneline
2020-01-28 [email protected] Revert "Always make gpu thread different from platform thread regardless of platform view (#16068)" (flutter/engine#16161)
2020-01-28 [email protected] Roll fuchsia/sdk/core/mac-amd64 from gNitp... to 5fMtM... (flutter/engine#16153)
2020-01-28 [email protected] Revert "Disable setting a library tag handler." (flutter/engine#16157)
2020-01-28 [email protected] Always make gpu thread different from platform thread regardless of platform view (flutter/engine#16068)
2020-01-28 [email protected] Add test to ensure that concurrent message loops have at least one workers. (flutter/engine#16074)
2020-01-28 [email protected] Revert breaking PRs (flutter/engine#16148)
2020-01-28 [email protected] Roll fuchsia/sdk/core/linux-amd64 from 8Ns10... to 2rLoq... (flutter/engine#15971)
2020-01-28 [email protected] Gets the DPI for all awareness mode and older Windows versions (flutter/engine#15951)
2020-01-28 [email protected] Hold a mutex when updating all CanPostTaskToAllNativeThreads::Captures members. (flutter/engine#16085)
2020-01-28 [email protected] Disable setting a library tag handler. (flutter/engine#16086)
2020-01-28 [email protected] Web PargraphStyle TextHeightBehavior integration (flutter/engine#16075)
2020-01-28 [email protected] Fix flake by making thread ID tracking in CanPostTaskToAllNativeThreads thread safe. (flutter/engine#16081)
2020-01-28 [email protected] Remove buggy assertion in EmbedderTest::CanPostTaskToAllNativeThreads. (flutter/engine#16071)
2020-01-27 [email protected] Remove tonic/platform. (flutter/engine#16062)
2020-01-27 [email protected] Allow embedders to schedule a callback on all engine managed threads. (flutter/engine#15980)
2020-01-27 [email protected] Engine/LibTxt/dart:ui impl of TextHeightBehavior (flutter/engine#15087)
2020-01-27 [email protected] Roll src/third_party/skia f1b2b42613cb..9c1d30dd163e (8 commits) (flutter/engine#16060)
2020-01-27 [email protected] Remove stale recipe changelog. (flutter/engine#15985)


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] on the revert to ensure that a human
is aware of the problem.

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/+/master/autoroll/README.md
@cyanglaz cyanglaz deleted the enable_thread_merging branch February 21, 2020 22:55
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
NoamDev pushed a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
NoamDev added a commit to NoamDev/engine that referenced this pull request Feb 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dynamically change thread configuration when a UIView is embedded
3 participants