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

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

Issue 1165853002: Pipe impl_latency_takes_priority_ to the RenderScheduler. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Made the flag default Created 5 years, 6 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: components/scheduler/renderer/renderer_scheduler_impl.cc
diff --git a/components/scheduler/renderer/renderer_scheduler_impl.cc b/components/scheduler/renderer/renderer_scheduler_impl.cc
index 9b168f83aeefbf5958a470ebc63d88b564c81bfa..0dc0a4f29f7d8b9fe276886f49da00f11ec2f23c 100644
--- a/components/scheduler/renderer/renderer_scheduler_impl.cc
+++ b/components/scheduler/renderer/renderer_scheduler_impl.cc
@@ -41,6 +41,7 @@ RendererSchedulerImpl::RendererSchedulerImpl(
was_shutdown_(false),
pending_main_thread_input_event_count_(0),
awaiting_touch_start_response_(false),
+ begin_main_frame_on_critical_path_(false),
last_input_type_(blink::WebInputEvent::Undefined),
policy_may_need_update_(&incoming_signals_lock_),
timer_queue_suspend_count_(0),
@@ -126,6 +127,7 @@ void RendererSchedulerImpl::WillBeginFrame(const cc::BeginFrameArgs& args) {
EndIdlePeriod();
estimated_next_frame_begin_ = args.frame_time + args.interval;
+ begin_main_frame_on_critical_path_ = args.on_critical_path;
}
void RendererSchedulerImpl::DidCommitFrameToCompositor() {
@@ -592,6 +594,8 @@ RendererSchedulerImpl::AsValueLocked(base::TimeTicks optional_now) const {
pending_main_thread_input_event_count_);
state->SetBoolean("awaiting_touch_start_response",
awaiting_touch_start_response_);
+ state->SetBoolean("begin_main_frame_on_critical_path",
+ begin_main_frame_on_critical_path_);
state->SetDouble(
"estimated_next_frame_begin",
(estimated_next_frame_begin_ - base::TimeTicks()).InMillisecondsF());

Powered by Google App Engine
This is Rietveld 408576698