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

Hold the TestRunner's platform view through a shared_ptr#3870

Merged
jason-simmons merged 1 commit intoflutter:masterfrom
jason-simmons:tester_weak_ptr
Jul 12, 2017
Merged

Hold the TestRunner's platform view through a shared_ptr#3870
jason-simmons merged 1 commit intoflutter:masterfrom
jason-simmons:tester_weak_ptr

Conversation

@jason-simmons
Copy link
Member

flutter_tester was throwing a bad_weak_ptr while setting up the engine
because the PlatformViewTest was held in a unique_ptr

flutter_tester was throwing a bad_weak_ptr while setting up the engine
because the PlatformViewTest was held in a unique_ptr
@jason-simmons
Copy link
Member Author

@B3rn475

@cbracken @chinmaygarde this fix may need to be ported over to iOS

goderbauer added a commit to goderbauer/engine that referenced this pull request Jul 12, 2017
@goderbauer goderbauer mentioned this pull request Jul 12, 2017
@goderbauer
Copy link
Member

For iOS: #3871

@jason-simmons jason-simmons merged commit a64921e into flutter:master Jul 12, 2017
goderbauer added a commit that referenced this pull request Jul 12, 2017
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