Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Don't do reentrant message channel calls. #560

Merged
merged 1 commit into from
May 9, 2018

Conversation

floitschG
Copy link
Contributor

Calling handlePlatformMessage while in a mockMethodCallHandler can
lead to race conditions since the first method call hasn't been fully
handled yet.

Calling `handlePlatformMessage` while in a mockMethodCallHandler can
lead to race conditions since the first method call hasn't been fully
handled yet.
Copy link
Contributor

@mravn-google mravn-google left a comment

Choose a reason for hiding this comment

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

LGTM

@floitschG floitschG merged commit c90c800 into flutter:master May 9, 2018
@floitschG floitschG deleted the fix_cloud_test branch May 9, 2018 20:03
slightfoot pushed a commit to slightfoot/plugins that referenced this pull request Jun 5, 2018
Calling `handlePlatformMessage` while in a mockMethodCallHandler can
lead to race conditions since the first method call hasn't been fully
handled yet.
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
Calling `handlePlatformMessage` while in a mockMethodCallHandler can
lead to race conditions since the first method call hasn't been fully
handled yet.
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.

3 participants