Chromium Code Reviews| Index: base/task_scheduler/scheduler_worker_pool_impl.h |
| diff --git a/base/task_scheduler/scheduler_worker_pool_impl.h b/base/task_scheduler/scheduler_worker_pool_impl.h |
| index 7b285a54d38a84d6380f97bc19e4945f414cae9d..58e158f7e6e4fe7daf91a3814b065ffcb31462d2 100644 |
| --- a/base/task_scheduler/scheduler_worker_pool_impl.h |
| +++ b/base/task_scheduler/scheduler_worker_pool_impl.h |
| @@ -38,16 +38,19 @@ namespace internal { |
| class DelayedTaskManager; |
| class TaskTracker; |
| -// A pool of workers that run Tasks. This class is thread-safe. |
| +// A pool of workers that run Tasks. |
| +// |
| +// The pool doesn't create threads before Start() is called. Tasks can be posted |
|
robliao
2017/04/24 22:32:19
Nit: before -> until
Apply to the other comments
fdoray
2017/04/25 18:43:34
Done.
|
| +// before Start() is called, but they don't run until Start() is called. |
|
robliao
2017/04/24 22:32:18
Tasks can be posted at any time but will not run u
fdoray
2017/04/25 18:43:34
Done.
|
| +// |
| +// This class is thread-safe. |
| class BASE_EXPORT SchedulerWorkerPoolImpl : public SchedulerWorkerPool { |
| public: |
| // Callback invoked when a Sequence isn't empty after a worker pops a Task |
| // from it. |
| using ReEnqueueSequenceCallback = Callback<void(scoped_refptr<Sequence>)>; |
| - // Constructs a pool without workers. Tasks can be posted to the pool, but |
| - // they won't run until workers are created. To create workers and start |
| - // running tasks, call Start(). |
| + // Constructs a pool without workers. |
| // |
| // |name| is used to label the pool's threads ("TaskScheduler" + |name| + |
| // index) and histograms ("TaskScheduler." + histogram name + "." + |name| + |