Worker: Introduce per-global-scope task scheduler

third_party/WebKit/Source/platform/scheduler/child/compositor_worker_scheduler.cc

 // 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.
 
 #include "platform/scheduler/child/compositor_worker_scheduler.h"
 
 #include <utility>
 
 #include "base/callback.h"
 #include "base/message_loop/message_loop.h"
 #include "base/threading/thread.h"
+#include "platform/scheduler/child/scheduler_helper.h"
+#include "platform/scheduler/child/scheduler_tqm_delegate.h"
 
 namespace blink {
 namespace scheduler {
 
-// TODO(scheduler-dev): Get rid of this asap!
-namespace {
-class CompositorWorkerTaskRunnerWrapper : public TaskQueue {
- public:
-  explicit CompositorWorkerTaskRunnerWrapper(
-      scoped_refptr<base::SingleThreadTaskRunner> task_runner)
-      : task_runner_(task_runner) {}
-
-  // TaskQueue implementation:
-  void UnregisterTaskQueue() override { NOTREACHED(); }
-
-  bool RunsTasksOnCurrentThread() const override {
-    return task_runner_->RunsTasksOnCurrentThread();
-  }
-
-  bool PostDelayedTask(const tracked_objects::Location& from_here,
-                       base::OnceClosure task,
-                       base::TimeDelta delay) override {
-    return task_runner_->PostDelayedTask(from_here, std::move(task), delay);
-  }
-
-  bool PostNonNestableDelayedTask(const tracked_objects::Location& from_here,
-                                  base::OnceClosure task,
-                                  base::TimeDelta delay) override {
-    return task_runner_->PostNonNestableDelayedTask(from_here, std::move(task),
-                                                    delay);
-  }
-
-  std::unique_ptr<QueueEnabledVoter> CreateQueueEnabledVoter() override {
-    NOTREACHED();
-    return nullptr;
-  }
-
-  void InsertFence(InsertFencePosition position) override { NOTREACHED(); }
-
-  void RemoveFence() override { NOTREACHED(); }
-
-  bool BlockedByFence() const override {
-    NOTREACHED();
-    return false;
-  }
-
-  bool IsQueueEnabled() const override {
-    NOTREACHED();
-    return true;
-  }
-
-  bool IsEmpty() const override {
-    NOTREACHED();
-    return false;
-  };
-
-  size_t GetNumberOfPendingTasks() const override {
-    NOTREACHED();
-    return 0;
-  };
-
-  bool HasPendingImmediateWork() const override {
-    NOTREACHED();
-    return false;
-  };
-
-  base::Optional<base::TimeTicks> GetNextScheduledWakeUp() override {
-    NOTREACHED();
-    return base::nullopt;
-  }
-
-  const char* GetName() const override {
-    NOTREACHED();
-    return nullptr;
-  };
-
-  QueueType GetQueueType() const override {
-    NOTREACHED();
-    return QueueType::DEFAULT;
-  }
-
-  void SetQueuePriority(QueuePriority priority) override { NOTREACHED(); }
-
-  QueuePriority GetQueuePriority() const override {
-    NOTREACHED();
-    return QueuePriority::NORMAL_PRIORITY;
-  };
-
-  void AddTaskObserver(
-      base::MessageLoop::TaskObserver* task_observer) override {
-    NOTREACHED();
-  }
-
-  void RemoveTaskObserver(
-      base::MessageLoop::TaskObserver* task_observer) override {
-    NOTREACHED();
-  }
-
-  void SetTimeDomain(TimeDomain* domain) override { NOTREACHED(); }
-
-  TimeDomain* GetTimeDomain() const override {
-    return nullptr;
-  }
-
-  void SetBlameContext(base::trace_event::BlameContext*) override {
-    NOTREACHED();
-  }
-
-  void SetObserver(Observer* observer) override { NOTREACHED(); }
-
- private:
-  ~CompositorWorkerTaskRunnerWrapper() override {}
-
-  scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
-};
-}  // namespace
-
-CompositorWorkerScheduler::CompositorWorkerScheduler(base::Thread* thread)
-    : thread_(thread) {}
+CompositorWorkerScheduler::CompositorWorkerScheduler(

[Sami, 2017/04/12 15:54:06]

FYI, last time we tried to do this there was a minor perf regression which we
mostly optimized away. The last thing remaining there is to add support for task
cancellation in base::MessageLoop. dcheng@ has a patch for that.

[nhiroki, 2017/04/13 08:23:50]

Thank you for the information. I'll take care of perf regression reports.

Should I wait for dcheng@'s patch before landing this?

[Sami, 2017/04/19 11:14:07]

It's probably fine to try landing it at least since we haven't measured in a
while. If the issue turns out to be huge we can improve as a follow-up.

[nhiroki, 2017/04/19 23:10:50]

OK, let's try!

+    base::Thread* thread,
+    scoped_refptr<SchedulerTqmDelegate> main_task_runner)
+    : thread_(thread),
+      helper_(new SchedulerHelper(
+          main_task_runner,
+          "compositor.scheduler",
+          TRACE_DISABLED_BY_DEFAULT("compositor.scheduler"),
+          TRACE_DISABLED_BY_DEFAULT("compositor.scheduler.debug"))) {}
 
 CompositorWorkerScheduler::~CompositorWorkerScheduler() {}
 
 void CompositorWorkerScheduler::Init() {}
 
 scoped_refptr<TaskQueue> CompositorWorkerScheduler::DefaultTaskRunner() {
   // TODO(sad): Implement a more robust scheduler that can do idle tasks for GC
   // without regressing performance of the rest of the system.
-  return make_scoped_refptr(
-      new CompositorWorkerTaskRunnerWrapper(thread_->task_runner()));
+  return helper_->DefaultTaskRunner();
 }
 
 scoped_refptr<scheduler::SingleThreadIdleTaskRunner>
 CompositorWorkerScheduler::IdleTaskRunner() {
   // TODO(flackr): This posts idle tasks as regular tasks. We need to create
   // an idle task runner with the semantics we want for the compositor thread
   // which runs them after the current frame has been drawn before the next
   // vsync. https://crbug.com/609532
   return make_scoped_refptr(new SingleThreadIdleTaskRunner(
       thread_->task_runner(), this, "compositor.scheduler"));
 }
 
 bool CompositorWorkerScheduler::CanExceedIdleDeadlineIfRequired() const {
   return false;
 }
 
 bool CompositorWorkerScheduler::ShouldYieldForHighPriorityWork() {
   return false;
 }
 
 void CompositorWorkerScheduler::AddTaskObserver(
     base::MessageLoop::TaskObserver* task_observer) {
-  thread_->message_loop()->AddTaskObserver(task_observer);
+  helper_->AddTaskObserver(task_observer);
 }
 
 void CompositorWorkerScheduler::RemoveTaskObserver(
     base::MessageLoop::TaskObserver* task_observer) {
-  thread_->message_loop()->RemoveTaskObserver(task_observer);
+  helper_->RemoveTaskObserver(task_observer);
 }
 
 void CompositorWorkerScheduler::Shutdown() {}
 
+scoped_refptr<TaskQueue> CompositorWorkerScheduler::NewUnthrottledTaskRunner(
+    TaskQueue::QueueType queue_type) {
+  helper_->CheckOnValidThread();
+  scoped_refptr<TaskQueue> unthrottled_task_queue(
+      helper_->NewTaskQueue(TaskQueue::Spec(queue_type)
+                                .SetShouldMonitorQuiescence(true)
+                                .SetTimeDomain(nullptr)));
+  return unthrottled_task_queue;
+}
+
 void CompositorWorkerScheduler::OnIdleTaskPosted() {}
 
 base::TimeTicks CompositorWorkerScheduler::WillProcessIdleTask() {
   // TODO(flackr): Return the next frame time as the deadline instead.
   // TODO(flackr): Ensure that oilpan GC does happen on the compositor thread
   // even though we will have no long idle periods. https://crbug.com/609531
   return base::TimeTicks::Now() + base::TimeDelta::FromMillisecondsD(16.7);
 }
 
 void CompositorWorkerScheduler::DidProcessIdleTask() {}
 
 base::TimeTicks CompositorWorkerScheduler::NowTicks() {
   return base::TimeTicks::Now();
 }
 
 }  // namespace scheduler
 }  // namespace blink