Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2481)

Unified Diff: base/task_scheduler/scheduler_worker.h

Issue 2399213005: TaskScheduler: Change Sequence::PeekTask to Sequence::TakeTask. (Closed)
Patch Set: self-review Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: base/task_scheduler/scheduler_worker.h
diff --git a/base/task_scheduler/scheduler_worker.h b/base/task_scheduler/scheduler_worker.h
index eee2f4a56c723304809cc99371c446f7bf64d6a3..a93a359fe35bd9ecc07de1d5a1e0fef12ed50005 100644
--- a/base/task_scheduler/scheduler_worker.h
+++ b/base/task_scheduler/scheduler_worker.h
@@ -53,9 +53,10 @@ class BASE_EXPORT SchedulerWorker {
// run a Task.
virtual scoped_refptr<Sequence> GetWork(SchedulerWorker* worker) = 0;
- // Called by the SchedulerWorker after it ran |task|. |task_latency| is the
- // time elapsed between when the task was posted and when it started to run.
- virtual void DidRunTask(const Task* task,
+ // Called by the SchedulerWorker after it ran a task with |task_priority|.
+ // |task_latency| is the time elapsed between when the task was posted and
+ // when it started to run.
+ virtual void DidRunTask(TaskPriority task_priority,
robliao 2016/10/07 20:57:11 Since we're no longer passing the task in, maybe a
fdoray 2016/10/11 12:29:19 Done.
const TimeDelta& task_latency) = 0;
// Called when |sequence| isn't empty after the SchedulerWorker pops a Task

Powered by Google App Engine
This is Rietveld 408576698