content: Rename RasterWorkerPool -> CategorizedWorkerPool.

content/renderer/categorized_worker_pool.h

 // Copyright 2015 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 CONTENT_RENDERER_RASTER_WORKER_POOL_H_
-#define CONTENT_RENDERER_RASTER_WORKER_POOL_H_
+#ifndef CONTENT_RENDERER_CATEGORIZED_WORKER_POOL_H_
+#define CONTENT_RENDERER_CATEGORIZED_WORKER_POOL_H_
 
 #include <memory>
 
 #include "base/callback.h"
 #include "base/containers/hash_tables.h"
 #include "base/macros.h"
 #include "base/sequenced_task_runner.h"
 #include "base/synchronization/condition_variable.h"
 #include "base/task_runner.h"
 #include "base/threading/simple_thread.h"
 #include "cc/raster/task_category.h"
 #include "cc/raster/task_graph_runner.h"
 #include "cc/raster/task_graph_work_queue.h"
 #include "content/common/content_export.h"
 
 namespace content {
 
-// A pool of threads used to run raster work.
-// Work can be scheduled on the threads using different interfaces.
-// The pool itself implements TaskRunner interface and tasks posted via that
-// interface might run in parallel.
-// CreateSequencedTaskRunner creates a sequenced task runner that might run in
-// parallel with other instances of sequenced task runners.
-// It's also possible to get the underlying TaskGraphRunner to schedule a graph
-// of tasks with their dependencies.
-class CONTENT_EXPORT RasterWorkerPool : public base::TaskRunner,
-                                        public cc::TaskGraphRunner {
+// A pool of threads used to run categorized work. The work can be scheduled on
+// the threads using different interfaces.
+// 1. The pool itself implements TaskRunner interface and tasks posted via that
+//    interface might run in parallel.
+// 2. CreateSequencedTaskRunner creates a sequenced task runner that might run
+//    in parallel with other instances of sequenced task runners.
+// 3. It's also possible to get the underlying TaskGraphRunner to schedule a

[reveman, 2016/05/31 16:05:47]

As TaskGraphRunner is now an abstract interface in the same way as TaskRunner,
can you update this comment to be similar to 1). E.g. "It also implements
TaskGraphRunner interface that allows a graph of tasks to be scheduled..."

[prashant.n, 2016/05/31 16:46:08]

May be earlier we had embedded object. I'll modify it.

+//    graph of tasks with their dependencies.
+class CONTENT_EXPORT CategorizedWorkerPool : public base::TaskRunner,
+                                             public cc::TaskGraphRunner {
  public:
-  RasterWorkerPool();
+  CategorizedWorkerPool();
 
   // Overridden from base::TaskRunner:
   bool PostDelayedTask(const tracked_objects::Location& from_here,
                        const base::Closure& task,
                        base::TimeDelta delay) override;
   bool RunsTasksOnCurrentThread() const override;
 
   // Overridden from cc::TaskGraphRunner:
   cc::NamespaceToken GetNamespaceToken() override;
   void ScheduleTasks(cc::NamespaceToken token, cc::TaskGraph* graph) override;
@@ skipping 17 matching lines @@
   // Once all the tasks are executed the method blocks until the threads are
   // terminated.
   void Shutdown();
 
   cc::TaskGraphRunner* GetTaskGraphRunner() { return this; }
 
   // Create a new sequenced task graph runner.
   scoped_refptr<base::SequencedTaskRunner> CreateSequencedTaskRunner();
 
  protected:
-  ~RasterWorkerPool() override;
+  ~CategorizedWorkerPool() override;
 
  private:
-  class RasterWorkerPoolSequencedTaskRunner;
-  friend class RasterWorkerPoolSequencedTaskRunner;
+  class CategorizedWorkerPoolSequencedTaskRunner;
+  friend class CategorizedWorkerPoolSequencedTaskRunner;
 
   // Simple Task for the TaskGraphRunner that wraps a closure.
   // This class is used to schedule TaskRunner tasks on the
   // |task_graph_runner_|.
   class ClosureTask : public cc::Task {
    public:
     explicit ClosureTask(const base::Closure& closure);
 
     // Overridden from cc::Task:
     void RunOnWorkerThread() override;
@@ skipping 50 matching lines @@
   base::ConditionVariable has_ready_to_run_background_tasks_cv_;
   // Condition variable that is waited on by origin threads until a namespace
   // has finished running all associated tasks.
   base::ConditionVariable has_namespaces_with_finished_running_tasks_cv_;
   // Set during shutdown. Tells Run() to return when no more tasks are pending.
   bool shutdown_;
 };
 
 }  // namespace content
 
-#endif  // CONTENT_RENDERER_RASTER_WORKER_POOL_H_
+#endif  // CONTENT_RENDERER_CATEGORIZED_WORKER_POOL_H_