Plumb activation time to main

cc/scheduler/scheduler.cc

 // Copyright 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/scheduler/scheduler.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/logging.h"
@@ skipping 74 matching lines @@
 }
 
 void Scheduler::SetCanDraw(bool can_draw) {
   state_machine_.SetCanDraw(can_draw);
   ProcessScheduledActions();
 }
 
 void Scheduler::NotifyReadyToActivate() {
   compositor_timing_history_->ReadyToActivate();
   state_machine_.NotifyReadyToActivate();
+  // Note the activate source-frame and timestamp
+  if (client_->SyncTreeSourceFrameNumber() != 0) {
+    ready_to_activate_time_.push_back(std::make_pair(

[brianderson, 2017/04/06 23:43:15]

emplace_back?

[panicker, 2017/04/08 00:29:06]

Done.

+        client_->SyncTreeSourceFrameNumber(), base::TimeTicks::Now()));
+    fprintf(stderr, "\n\t\tScheduler::NotifyReadyToActivate: ptree: %d",

[vmpstr, 2017/04/05 19:11:33]

this is just for debugging I assume?

[panicker, 2017/04/08 00:29:06]

Yes I plan to clean these up before submitting

+            client_->SyncTreeSourceFrameNumber());
+  }
   ProcessScheduledActions();
 }
 
 void Scheduler::NotifyReadyToDraw() {
   // Future work might still needed for crbug.com/352894.
   state_machine_.NotifyReadyToDraw();
   ProcessScheduledActions();
 }
 
 void Scheduler::SetBeginFrameSource(BeginFrameSource* source) {
@@ skipping 515 matching lines @@
         &inside_action_, action);
     switch (action) {
       case SchedulerStateMachine::ACTION_NONE:
         break;
       case SchedulerStateMachine::ACTION_SEND_BEGIN_MAIN_FRAME:
         compositor_timing_history_->WillBeginMainFrame(
             begin_main_frame_args_.on_critical_path,
             begin_main_frame_args_.frame_time);
         state_machine_.WillSendBeginMainFrame();
         // TODO(brianderson): Pass begin_main_frame_args_ directly to client.
+        begin_main_frame_args_.ready_to_activate_time = ready_to_activate_time_;

[vmpstr, 2017/04/05 19:11:33]

std::move

[panicker, 2017/04/08 00:29:06]

Done.

+        ready_to_activate_time_.clear();
         client_->ScheduledActionSendBeginMainFrame(begin_main_frame_args_);
         break;
       case SchedulerStateMachine::ACTION_COMMIT: {
         bool commit_has_no_updates = false;
         state_machine_.WillCommit(commit_has_no_updates);
         client_->ScheduledActionCommit();
         break;
       }
       case SchedulerStateMachine::ACTION_ACTIVATE_SYNC_TREE:
         compositor_timing_history_->WillActivate();
@@ skipping 159 matching lines @@
 }
 
 BeginFrameAck Scheduler::CurrentBeginFrameAckForActiveTree() const {
   return BeginFrameAck(
       begin_main_frame_args_.source_id, begin_main_frame_args_.sequence_number,
       state_machine_.last_begin_frame_sequence_number_active_tree_was_fresh(),
       true);
 }
 
 }  // namespace cc