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

In tests run dart code on ui(rather than on platform) thread.#17686

Merged
fluttergithubbot merged 4 commits intoflutter:masterfrom
aam:run-test-on-ui
Apr 13, 2020
Merged

In tests run dart code on ui(rather than on platform) thread.#17686
fluttergithubbot merged 4 commits intoflutter:masterfrom
aam:run-test-on-ui

Conversation

@aam
Copy link
Member

@aam aam commented Apr 13, 2020

Copy link
Contributor

@chinmaygarde chinmaygarde left a comment

Choose a reason for hiding this comment

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

Wont this mean you are preparing the isolate on one thread but running it on another thread? If anything, the entirety of RunDartCodeInIsolate should run on the UI isolate.

@aam
Copy link
Member Author

aam commented Apr 13, 2020

Wont this mean you are preparing the isolate on one thread but running it on another thread? If anything, the entirety of RunDartCodeInIsolate should run on the UI isolate.

True, fixed.

@aam aam added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Apr 13, 2020
@fluttergithubbot fluttergithubbot merged commit 053da38 into flutter:master Apr 13, 2020
goderbauer pushed a commit to goderbauer/engine that referenced this pull request Apr 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MultiFrameImageCodec test fails with multiple mutators error

4 participants