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

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: Even moar tests, add a null scheduler and follow review. 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..e380e5d987b9232360ac2cf7517a485fe805d917
--- /dev/null
+++ b/content/renderer/scheduler/renderer_scheduler_impl.cc
@@ -0,0 +1,163 @@
+// 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_scheduler_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_scheduler_selector_(new RendererSchedulerSelector()),
+ task_queue_manager_(kTaskQueueCount,
+ main_task_runner,
+ renderer_scheduler_selector_.get()),
+ control_task_runner_(
+ task_queue_manager_.TaskRunnerForQueue(kControlTaskQueue)),
+ policy_may_need_update_(0),
+ current_policy_(kNormalPriorityPolicy),
+ weak_factory_(this) {
+ weak_renderer_scheduler_ptr_ = weak_factory_.GetWeakPtr();
+ idle_task_runner_ = make_scoped_refptr(new SingleThreadIdleTaskRunner(
+ task_queue_manager_.TaskRunnerForQueue(kIdleTaskQueue),
+ weak_factory_.GetWeakPtr()));
+ renderer_scheduler_selector_->SetQueuePriority(
+ kControlTaskQueue, RendererSchedulerSelector::kControlPriority);
+ renderer_scheduler_selector_->DisableQueue(kIdleTaskQueue);
+ task_queue_manager_.SetAutoPump(kIdleTaskQueue, false);
+}
+
+RendererSchedulerImpl::~RendererSchedulerImpl() {
+}
+
+scoped_refptr<base::SingleThreadTaskRunner>
+RendererSchedulerImpl::DefaultTaskRunner() {
+ return task_queue_manager_.TaskRunnerForQueue(kDefaultTaskQueue);
+}
+
+scoped_refptr<base::SingleThreadTaskRunner>
+RendererSchedulerImpl::CompositorTaskRunner() {
+ return task_queue_manager_.TaskRunnerForQueue(kCompositorTaskQueue);
+}
+
+scoped_refptr<SingleThreadIdleTaskRunner>
+RendererSchedulerImpl::IdleTaskRunner() {
+ return idle_task_runner_;
+}
+
+void RendererSchedulerImpl::WillBeginFrame(const cc::BeginFrameArgs& args) {
+ main_thread_checker_.CalledOnValidThread();
+ EndIdlePeriod();
+ estimated_next_frame_begin_ = args.frame_time + args.interval;
+}
+
+void RendererSchedulerImpl::DidCommitFrameToCompositor() {
+ main_thread_checker_.CalledOnValidThread();
+ if (Now() < estimated_next_frame_begin_) {
+ StartIdlePeriod();
+ }
+}
+
+void RendererSchedulerImpl::DidReceiveInputEvent() {
+ 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();
+ return SchedulerPolicy() == kCompositorPriorityPolicy &&
+ !task_queue_manager_.IsQueueEmpty(kCompositorTaskQueue);
+}
+
+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);
+ }
+}
+
+void RendererSchedulerImpl::UpdatePolicy() {
+ main_thread_checker_.CalledOnValidThread();
+ base::AutoLock lock(incoming_signals_lock_);
+ base::subtle::Release_Store(&policy_may_need_update_, 0);
+
+ Policy new_policy = kNormalPriorityPolicy;
+ 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 = kCompositorPriorityPolicy;
+ } 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_) {
alexclarke 2014/10/24 15:56:29 Consider adding: TRACE_COUNTER1(TRACE_DISABLED_BY
Sami 2014/10/24 15:58:21 I've got a follow-up that adds all sorts of tracin
+ return;
+ }
+
+ switch (new_policy) {
+ case kCompositorPriorityPolicy:
+ renderer_scheduler_selector_->SetQueuePriority(
+ kCompositorTaskQueue, RendererSchedulerSelector::kHighPriority);
+ break;
+ case kNormalPriorityPolicy:
+ renderer_scheduler_selector_->SetQueuePriority(
+ kCompositorTaskQueue, RendererSchedulerSelector::kNormalPriority);
+ break;
+ }
+ current_policy_ = new_policy;
+}
+
+void RendererSchedulerImpl::StartIdlePeriod() {
+ renderer_scheduler_selector_->EnableQueue(
+ kIdleTaskQueue, RendererSchedulerSelector::kBestEffortPriority);
+ task_queue_manager_.PumpQueue(kIdleTaskQueue);
+}
+
+void RendererSchedulerImpl::EndIdlePeriod() {
+ renderer_scheduler_selector_->DisableQueue(kIdleTaskQueue);
+}
+
+base::TimeTicks RendererSchedulerImpl::Now() const {
+ return gfx::FrameTime::Now();
+}
+
+} // namespace content

Powered by Google App Engine
This is Rietveld 408576698