New issue
Advanced search Search tips

Issue 662191 link

Starred by 2 users
Status: Fixed
Owner:
gab@chromium.org
Closed: Nov 2016
Cc:
robliao@chromium.org
fdoray@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 653916



Sign in to add a comment

SchedulerSingleThreadTaskRunner:RunsTasksOnCurrentThread should take advantage of SequenceToken

Project Member Reported by gab@chromium.org, Nov 3 2016 
Just like ThreadChecker does. Being assigned to the same thread shouldn't be sufficient, actually having been posted through this SchedulerSingleThreadTaskRunner is what matters.
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 7 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/cdac39241efc49fadf95a13df932f1f1e1eab93d

commit cdac39241efc49fadf95a13df932f1f1e1eab93d
Author: gab <gab@chromium.org>
Date: Mon Nov 07 20:40:25 2016

Make SchedulerSingleThreadTaskRunner::RunsTasksOnCurrentThread sequence-affine.

Added test highlights the requirements (fails with previous impl).

BUG= 662191 

Review-Url: https://codereview.chromium.org/2480613002
Cr-Commit-Position: refs/heads/master@{#430364}

[modify] https://crrev.com/cdac39241efc49fadf95a13df932f1f1e1eab93d/base/task_scheduler/scheduler_worker_pool_impl.cc
[modify] https://crrev.com/cdac39241efc49fadf95a13df932f1f1e1eab93d/base/task_scheduler/scheduler_worker_pool_impl_unittest.cc

Comment 2 by gab@chromium.org, Nov 7 2016

Status: Fixed (was: Started)
►

Sign in to add a comment