Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1357)

Unified Diff: content/renderer/scheduler/renderer_scheduler_impl.cc

Issue 664963002: content: Add RendererScheduler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: RendererSchedulerSelector -> RendererTaskQueueSelector Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/renderer/scheduler/renderer_scheduler_impl.cc
diff --git a/content/renderer/scheduler/renderer_scheduler_impl.cc b/content/renderer/scheduler/renderer_scheduler_impl.cc
new file mode 100644
index 0000000000000000000000000000000000000000..767f942dfb5e2f043b3159f179ef29c9fdff6725
--- /dev/null
+++ b/content/renderer/scheduler/renderer_scheduler_impl.cc
@@ -0,0 +1,186 @@
+// Copyright 2014 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 "content/renderer/scheduler/renderer_scheduler_impl.h"
+
+#include "base/bind.h"
+#include "base/message_loop/message_loop_proxy.h"
+#include "cc/output/begin_frame_args.h"
+#include "content/renderer/scheduler/renderer_task_queue_selector.h"
+#include "ui/gfx/frame_time.h"
+
+namespace content {
+
+RendererSchedulerImpl::RendererSchedulerImpl()
+ : RendererSchedulerImpl(base::MessageLoopProxy::current()) {
+}
+
+RendererSchedulerImpl::RendererSchedulerImpl(
+ scoped_refptr<base::SingleThreadTaskRunner> main_task_runner)
+ : renderer_task_queue_selector_(new RendererTaskQueueSelector()),
+ task_queue_manager_(
+ new TaskQueueManager(TASK_QUEUE_COUNT,
+ main_task_runner,
+ renderer_task_queue_selector_.get())),
+ control_task_runner_(
+ task_queue_manager_->TaskRunnerForQueue(CONTROL_TASK_QUEUE)),
+ default_task_runner_(
+ task_queue_manager_->TaskRunnerForQueue(DEFAULT_TASK_QUEUE)),
+ compositor_task_runner_(
+ task_queue_manager_->TaskRunnerForQueue(COMPOSITOR_TASK_QUEUE)),
+ current_policy_(NORMAL_PRIORITY_POLICY),
+ policy_may_need_update_(0),
+ weak_factory_(this) {
+ weak_renderer_scheduler_ptr_ = weak_factory_.GetWeakPtr();
+ idle_task_runner_ = make_scoped_refptr(new SingleThreadIdleTaskRunner(
+ task_queue_manager_->TaskRunnerForQueue(IDLE_TASK_QUEUE),
+ weak_factory_.GetWeakPtr()));
+ renderer_task_queue_selector_->SetQueuePriority(
+ CONTROL_TASK_QUEUE, RendererTaskQueueSelector::CONTROL_PRIORITY);
+ renderer_task_queue_selector_->DisableQueue(IDLE_TASK_QUEUE);
+ task_queue_manager_->SetAutoPump(IDLE_TASK_QUEUE, false);
+}
+
+RendererSchedulerImpl::~RendererSchedulerImpl() {
+}
+
+void RendererSchedulerImpl::Shutdown() {
+ task_queue_manager_.reset();
+}
+
+scoped_refptr<base::SingleThreadTaskRunner>
+RendererSchedulerImpl::DefaultTaskRunner() {
+ return default_task_runner_;
+}
+
+scoped_refptr<base::SingleThreadTaskRunner>
+RendererSchedulerImpl::CompositorTaskRunner() {
+ return compositor_task_runner_;
+}
+
+scoped_refptr<SingleThreadIdleTaskRunner>
+RendererSchedulerImpl::IdleTaskRunner() {
+ return idle_task_runner_;
+}
+
+void RendererSchedulerImpl::WillBeginFrame(const cc::BeginFrameArgs& args) {
+ main_thread_checker_.CalledOnValidThread();
+ if (!task_queue_manager_)
+ return;
+
+ EndIdlePeriod();
+ estimated_next_frame_begin_ = args.frame_time + args.interval;
+}
+
+void RendererSchedulerImpl::DidCommitFrameToCompositor() {
+ main_thread_checker_.CalledOnValidThread();
+ if (!task_queue_manager_)
+ return;
+
+ if (Now() < estimated_next_frame_begin_) {
+ StartIdlePeriod();
+ }
+}
+
+void RendererSchedulerImpl::DidReceiveInputEvent() {
+ if (!task_queue_manager_)
+ return;
+ base::AutoLock lock(incoming_signals_lock_);
+ if (last_input_time_.is_null()) {
+ // Update scheduler policy if should start a new compositor policy mode.
+ base::subtle::Release_Store(&policy_may_need_update_, 1);
+ PostUpdatePolicyOnControlRunner(base::TimeDelta());
+ }
+ last_input_time_ = Now();
+}
+
+bool RendererSchedulerImpl::ShouldYieldForHighPriorityWork() {
+ main_thread_checker_.CalledOnValidThread();
+ if (!task_queue_manager_)
+ return false;
+
+ return SchedulerPolicy() == COMPOSITOR_PRIORITY_POLICY &&
+ !task_queue_manager_->IsQueueEmpty(COMPOSITOR_TASK_QUEUE);
+}
+
+base::TimeTicks RendererSchedulerImpl::CurrentIdleTaskDeadline() const {
+ main_thread_checker_.CalledOnValidThread();
+ return estimated_next_frame_begin_;
+}
+
+RendererSchedulerImpl::Policy RendererSchedulerImpl::SchedulerPolicy() {
+ main_thread_checker_.CalledOnValidThread();
+ if (base::subtle::Acquire_Load(&policy_may_need_update_)) {
+ UpdatePolicy();
+ }
+ return current_policy_;
+}
+
+void RendererSchedulerImpl::PostUpdatePolicyOnControlRunner(
+ base::TimeDelta delay) {
+ base::Closure closure = base::Bind(&RendererSchedulerImpl::UpdatePolicy,
+ weak_renderer_scheduler_ptr_);
+ if (delay > base::TimeDelta()) {
+ control_task_runner_->PostDelayedTask(FROM_HERE, closure, delay);
+ } else {
+ control_task_runner_->PostTask(FROM_HERE, closure);
Sami 2014/10/29 13:51:27 nit: PostTask just calls PostDelayedTask with a ze
rmcilroy 2014/10/29 15:01:26 Done.
+ }
+}
+
+void RendererSchedulerImpl::UpdatePolicy() {
+ main_thread_checker_.CalledOnValidThread();
+ if (!task_queue_manager_)
+ return;
+
+ base::AutoLock lock(incoming_signals_lock_);
+ base::subtle::Release_Store(&policy_may_need_update_, 0);
+
+ Policy new_policy = NORMAL_PRIORITY_POLICY;
+ if (!last_input_time_.is_null()) {
+ base::TimeDelta compositor_priority_duration =
+ base::TimeDelta::FromMilliseconds(kCompositorPriorityAfterTouchMillis);
+ base::TimeDelta remaining_compositor_priority_duration =
+ last_input_time_ + compositor_priority_duration - Now();
+ if (remaining_compositor_priority_duration > base::TimeDelta()) {
+ PostUpdatePolicyOnControlRunner(remaining_compositor_priority_duration);
+ new_policy = COMPOSITOR_PRIORITY_POLICY;
+ } else {
+ // Null out last_input_time_ to ensure DidReceiveInputEvent will post an
+ // UpdatePolicy task when it's next called.
+ last_input_time_ = base::TimeTicks();
+ }
+ }
+
+ if (new_policy == current_policy_) {
+ return;
+ }
+
+ switch (new_policy) {
+ case COMPOSITOR_PRIORITY_POLICY:
+ renderer_task_queue_selector_->SetQueuePriority(
+ COMPOSITOR_TASK_QUEUE, RendererTaskQueueSelector::HIGH_PRIORITY);
+ break;
+ case NORMAL_PRIORITY_POLICY:
+ renderer_task_queue_selector_->SetQueuePriority(
+ COMPOSITOR_TASK_QUEUE, RendererTaskQueueSelector::NORMAL_PRIORITY);
+ break;
+ }
+ current_policy_ = new_policy;
+}
+
+void RendererSchedulerImpl::StartIdlePeriod() {
+ renderer_task_queue_selector_->EnableQueue(
+ IDLE_TASK_QUEUE, RendererTaskQueueSelector::BEST_EFFORT_PRIORITY);
+ task_queue_manager_->PumpQueue(IDLE_TASK_QUEUE);
+}
+
+void RendererSchedulerImpl::EndIdlePeriod() {
+ renderer_task_queue_selector_->DisableQueue(IDLE_TASK_QUEUE);
+}
+
+base::TimeTicks RendererSchedulerImpl::Now() const {
+ return gfx::FrameTime::Now();
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698