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

Unified Diff: cc/scheduler/scheduler.cc

Issue 1101603003: HACK: Drop frame rate of Renderer. Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 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
« no previous file with comments | « cc/scheduler/scheduler.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/scheduler/scheduler.cc
diff --git a/cc/scheduler/scheduler.cc b/cc/scheduler/scheduler.cc
index 2288169ecde077c80dc44fe911097a87924f1869..ded29bfbc3b3d6831f99b89077b9e0af3c8c9647 100644
--- a/cc/scheduler/scheduler.cc
+++ b/cc/scheduler/scheduler.cc
@@ -66,7 +66,8 @@ Scheduler::Scheduler(
const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
scoped_ptr<BeginFrameSource> external_begin_frame_source,
SchedulerFrameSourcesConstructor* frame_sources_constructor)
- : frame_source_(),
+ : begin_frame_count_(0),
+ frame_source_(),
primary_frame_source_(NULL),
primary_frame_source_internal_(external_begin_frame_source.Pass()),
vsync_observer_(NULL),
@@ -340,6 +341,15 @@ void Scheduler::SetupPollingMechanisms() {
bool Scheduler::OnBeginFrameMixInDelegate(const BeginFrameArgs& args) {
TRACE_EVENT1("cc,benchmark", "Scheduler::BeginFrame", "args", args.AsValue());
+ bool in_renderer = !settings_.main_thread_should_always_be_low_latency;
+ if (in_renderer) {
+ const int kFramesToSkip = 1;
+ begin_frame_count_++;
+ if (begin_frame_count_ <= kFramesToSkip)
+ return false;
+ begin_frame_count_ = 0;
+ }
+
// Deliver BeginFrames to children.
if (state_machine_.children_need_begin_frames()) {
BeginFrameArgs adjusted_args_for_children(args);
« no previous file with comments | « cc/scheduler/scheduler.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698