Chromium Code Reviews
DescriptionRemove call to IsRunningSequenceOnCurrentThread() from api_resource_manager.h.
SequencedWorkerPool is being migrated to base/task_scheduler and there
is no equivalent to IsRunningSequenceOnCurrentThread() in
base/task_scheduler.
We believe that always doing work asynchronously is cleaner than using
this pattern:
if (pool->IsRunningSequencedOnCurrentThread(token))
DoSequencedWork();
else
pool->PostSequencedWorkerTask(token, FROM_HERE, Bind(&DoSequencedWork));
Developers should us SequenceChecker to verify that tasks run
sequentially. Unlike
SequencedWorkerPool::IsRunningSequenceOnCurrentThread(),
SequenceChecker works everywhere in Chrome (MessageLoop,
SequencedWorkerPool, base/task_scheduler...).
BUG=622400
Committed: https://crrev.com/62f47315fa2bae98943888334e2787ce308659e8
Cr-Commit-Position: refs/heads/master@{#416679}
Patch Set 1 #
Messages
Total messages: 12 (7 generated)
|
|||||||||||||||||||