Introduce a SchedulerHelper in content/child/scheduler

content/child/scheduler/task_queue_selector_impl.h

Index: content/child/scheduler/task_queue_selector_impl.h
diff --git a/content/renderer/scheduler/renderer_task_queue_selector.h b/content/child/scheduler/task_queue_selector_impl.h
similarity index 85%
rename from content/renderer/scheduler/renderer_task_queue_selector.h
rename to content/child/scheduler/task_queue_selector_impl.h
index 72bbc62874047085e99102f29da4c73b47bad2e0..c59a506541a63ab792228211010c2456cbf66213 100644
--- a/content/renderer/scheduler/renderer_task_queue_selector.h
+++ b/content/child/scheduler/task_queue_selector_impl.h
@@ -2,21 +2,21 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CONTENT_RENDERER_SCHEDULER_RENDERER_TASK_QUEUE_SELECTOR_H_
-#define CONTENT_RENDERER_SCHEDULER_RENDERER_TASK_QUEUE_SELECTOR_H_
+#ifndef CONTENT_CHILD_SCHEDULER_TASK_QUEUE_SELECTOR_IMPL_H_
+#define CONTENT_CHILD_SCHEDULER_TASK_QUEUE_SELECTOR_IMPL_H_
 
 #include <set>
 
 #include "base/compiler_specific.h"
 #include "base/threading/thread_checker.h"
+#include "content/child/scheduler/task_queue_selector.h"
 #include "content/common/content_export.h"
-#include "content/renderer/scheduler/task_queue_selector.h"
 
 namespace content {
 
-// A RendererTaskQueueSelector is a TaskQueueSelector which is used by the
-// RendererScheduler to enable prioritization of particular task queues.
-class CONTENT_EXPORT RendererTaskQueueSelector
+// A TaskQueueSelectorImpl is a TaskQueueSelector which is used by the
+// SchedulerHelper to enable prioritization of particular task queues.
+class CONTENT_EXPORT TaskQueueSelectorImpl

[Sami, 2015/03/27 01:15:55]

nit: Instead of "Impl" let's call this something like
PrioritizingTaskQueueSelector to make its function clearer and to make it more
obvious why we need a different one for workers.

[alex clarke (OOO till 29th), 2015/03/27 10:55:29]

So PrioritizingTaskQueueSelector does sound like a better name. However I intend
to use PrioritizingTaskQueueSelector for workers too.

     : NON_EXPORTED_BASE(public TaskQueueSelector) {
  public:
   enum QueuePriority {
@@ -38,8 +38,8 @@ class CONTENT_EXPORT RendererTaskQueueSelector
     FIRST_QUEUE_PRIORITY = CONTROL_PRIORITY,
   };
 
-  RendererTaskQueueSelector();
-  ~RendererTaskQueueSelector() override;
+  TaskQueueSelectorImpl();
+  ~TaskQueueSelectorImpl() override;
 
   // Set the priority of |queue_index| to |priority|.
   void SetQueuePriority(size_t queue_index, QueuePriority priority);
@@ -93,9 +93,9 @@ class CONTENT_EXPORT RendererTaskQueueSelector
   std::vector<const base::TaskQueue*> work_queues_;
   std::set<size_t> queue_priorities_[QUEUE_PRIORITY_COUNT];
   size_t starvation_count_;
-  DISALLOW_COPY_AND_ASSIGN(RendererTaskQueueSelector);
+  DISALLOW_COPY_AND_ASSIGN(TaskQueueSelectorImpl);
 };
 
 }  // namespace content
 
-#endif  // CONTENT_RENDERER_SCHEDULER_RENDERER_TASK_QUEUE_SELECTOR_H_
+#endif  // CONTENT_CHILD_SCHEDULER_TASK_QUEUE_SELECTOR_IMPL_H_