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

Unified Diff: cc/surfaces/display_scheduler.cc

Issue 1821863002: Hook up ui::Compositor to Display's BeginFrameSource (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: BrowserCompositorOutputSurface owns;scheduler decides Created 4 years, 9 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: cc/surfaces/display_scheduler.cc
diff --git a/cc/surfaces/display_scheduler.cc b/cc/surfaces/display_scheduler.cc
index 28f780862bc10c93388c09b76d6c45f93556fc51..9fb863cb1f8e527b491130dfe543170b5b4e3019 100644
--- a/cc/surfaces/display_scheduler.cc
+++ b/cc/surfaces/display_scheduler.cc
@@ -34,9 +34,12 @@ DisplayScheduler::DisplayScheduler(DisplaySchedulerClient* client,
begin_frame_deadline_closure_ = base::Bind(
&DisplayScheduler::OnBeginFrameDeadline, weak_ptr_factory_.GetWeakPtr());
- // TODO(tansell): Set this to something useful.
+ // TODO(enne): remove this once the renderer uses begin frames everywhere.
begin_frame_source_for_children_ = SyntheticBeginFrameSource::Create(
task_runner, BeginFrameArgs::DefaultInterval());
+
+ // TODO(enne): For now, this is always the same, so just set initially.
+ client_->UpdateSchedulerBeginFrameSource(begin_frame_source_);
enne (OOO) 2016/03/25 17:55:36 I'm not sure about the split between DisplaySchedu
}
DisplayScheduler::~DisplayScheduler() {

Powered by Google App Engine
This is Rietveld 408576698