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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: cc/trees/proxy_main.cc

Issue 2778223005: Plumb activation time to main (Closed)
Patch Set: add a vector instead of single activate frame time 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/trees/proxy_impl.cc ('K') | « cc/trees/proxy_impl.cc ('k') | cc/trees/single_thread_proxy.h » ('j') | cc/trees/single_thread_proxy.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/proxy_main.cc
diff --git a/cc/trees/proxy_main.cc b/cc/trees/proxy_main.cc
index de79051d4b38e3395e219f1054e431150859bd37..1acb9d37d5cf039fd07400b3f546699508d6630a 100644
--- a/cc/trees/proxy_main.cc
+++ b/cc/trees/proxy_main.cc
@@ -118,6 +118,10 @@ void ProxyMain::BeginMainFrame(
benchmark_instrumentation::kDoBeginFrame,
begin_main_frame_state->begin_frame_id);
+ // Set frame_source_number in BeginFrameArgs
vmpstr 2017/04/05 19:11:33 I don't think you need this comment :)
panicker 2017/04/08 00:29:07 Done.
+ begin_main_frame_state->begin_frame_args.frame_source_number =
+ layer_tree_host_->SourceFrameNumber();
+
base::TimeTicks begin_main_frame_start_time = base::TimeTicks::Now();
TRACE_EVENT_SYNTHETIC_DELAY_BEGIN("cc.BeginMainFrame");
« cc/trees/proxy_impl.cc ('K') | « cc/trees/proxy_impl.cc ('k') | cc/trees/single_thread_proxy.h » ('j') | cc/trees/single_thread_proxy.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698