TaskScheduler [11] Support ExecutionMode::SINGLE_THREADED.

base/task_scheduler/scheduler_thread_pool.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef BASE_TASK_SCHEDULER_SCHEDULER_THREAD_POOL_H_
 #define BASE_TASK_SCHEDULER_SCHEDULER_THREAD_POOL_H_
 
 #include <memory>
 
 #include "base/base_export.h"
 #include "base/memory/ref_counted.h"
 #include "base/task_runner.h"
 #include "base/task_scheduler/sequence.h"
 #include "base/task_scheduler/task.h"
 #include "base/task_scheduler/task_traits.h"
 
 namespace base {
 namespace internal {
 
+class SchedulerWorkerThread;
 struct SequenceSortKey;
 
 // Interface for a thread pool.
 class BASE_EXPORT SchedulerThreadPool {
  public:
   virtual ~SchedulerThreadPool() = default;
 
   // Returns a TaskRunner whose PostTask invocations will result in scheduling
   // Tasks with |traits| and |execution_mode| in this thread pool.
   virtual scoped_refptr<TaskRunner> CreateTaskRunnerWithTraits(
       const TaskTraits& traits,
       ExecutionMode execution_mode) = 0;
 
   // Inserts |sequence| into this thread pool's shared priority queue with
   // |sequence_sort_key|. Must only be called from a worker thread to put
   // |sequence| back into a PriorityQueue after running a Task from it. The
   // worker thread doesn't have to belong to this thread pool.
   virtual void ReEnqueueSequence(scoped_refptr<Sequence> sequence,
                                  const SequenceSortKey& sequence_sort_key) = 0;
 
-  // Posts |task| to be executed as part of |sequence|.  Returns true if |task|
-  // is posted.
+  // Posts |task| to be executed as part of |sequence|. |task| will run on
+  // |worker_thread| if specified (must be owned by this thread pool).  Returns

[gab, 2016/04/26 11:46:27]

As-is it's unclear whether the comments in parens refers to |task| or
|worker_thread| IMO. How about:

"If |worker_thread| is non-null, |task| will be scheduled to run on it
specifically (note: |worker_thread| must be owned by this SchedulerThreadPool);
otherwise, |task| will be added to the pending shared work"

[fdoray, 2016/04/26 14:56:47]

Done.

+  // true if |task| is posted.
   virtual bool PostTaskWithSequence(std::unique_ptr<Task> task,
-                                    scoped_refptr<Sequence> sequence) = 0;
+                                    scoped_refptr<Sequence> sequence,
+                                    SchedulerWorkerThread* worker_thread) = 0;
 
-  // Posts |task| to be executed by this thread pool as part of |sequence|. The
-  // scheduler's TaskTracker must have allowed |task| to be posted before this
-  // is called. This must only be called after |task|'s delayed run time.
-  virtual void PostTaskWithSequenceNow(std::unique_ptr<Task> task,
-                                       scoped_refptr<Sequence> sequence) = 0;
+  // Posts |task| to be executed by this thread pool as part of |sequence|.
+  // |task| will run on |worker_thread| if specified (must be owned by this
+  // thread pool). The scheduler's TaskTracker must have allowed |task| to be
+  // posted before this is called. This must only be called after |task|'s
+  // delayed run time.
+  virtual void PostTaskWithSequenceNow(
+      std::unique_ptr<Task> task,
+      scoped_refptr<Sequence> sequence,
+      SchedulerWorkerThread* worker_thread) = 0;
 };
 
 }  // namespace internal
 }  // namespace base
 
 #endif  // BASE_TASK_SCHEDULER_SCHEDULER_THREAD_POOL_H_