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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/trees/single_thread_proxy.cc

Issue 2778223005: Plumb activation time to main (Closed)
Patch Set: pass 0 for sync tree in NotifyReadyToActivate from SingleThreadProxy Created 3 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
« cc/scheduler/scheduler.cc ('K') | « cc/trees/proxy_main.cc ('k') | third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/single_thread_proxy.cc
diff --git a/cc/trees/single_thread_proxy.cc b/cc/trees/single_thread_proxy.cc
index 0386145c7c55fea08e4f34362dd5f3536b1b1a02..2e7a4fbc75a7692d03111f0a0c087259efb2100f 100644
--- a/cc/trees/single_thread_proxy.cc
+++ b/cc/trees/single_thread_proxy.cc
@@ -319,8 +319,11 @@ void SingleThreadProxy::OnCanDrawStateChanged(bool can_draw) {
void SingleThreadProxy::NotifyReadyToActivate() {
TRACE_EVENT0("cc", "SingleThreadProxy::NotifyReadyToActivate");
DebugScopedSetImplThread impl(task_runner_provider_);
- if (scheduler_on_impl_thread_)
- scheduler_on_impl_thread_->NotifyReadyToActivate();
+ if (scheduler_on_impl_thread_) {
+ // The argument to NotifyReadyToActivate is the pending-tree for computing
+ // Activation time in ProxyImpl. However this is not applicable here.
+ scheduler_on_impl_thread_->NotifyReadyToActivate(0);
+ }
}
void SingleThreadProxy::NotifyReadyToDraw() {
« cc/scheduler/scheduler.cc ('K') | « cc/trees/proxy_main.cc ('k') | third_party/WebKit/Source/platform/scheduler/renderer/renderer_scheduler_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698