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

Conversation

cyanglaz
Copy link
Contributor

The TaskRunnerAffineWeakPtr was introduced in #17649.
At the time, A TaskRunnerAffineWeakPtr is generated by WeakPtrFactory. The WeakPtrFactory can only be accessed from a single thread.
This pr makes the factory also accessible from different threads (within a single task runner)

The PR also lowers the access level of some fields in the WeakPtr

Copy link
Contributor

@iskakaushik iskakaushik 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 9ce1e5c into flutter:master May 19, 2020
@cyanglaz cyanglaz deleted the task_runner_affine_factory branch May 19, 2020 23:36
engine-flutter-autoroll added a commit to engine-flutter-autoroll/flutter that referenced this pull request May 20, 2020
wandyers pushed a commit to wandyers/engine that referenced this pull request May 23, 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.

3 participants