[cc] Add and use BeginFrameAck for DidFinishFrame.

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 304 matching lines @@
   BeginFrameArgs adjusted_args = args;
   // Cancel the missed begin frame task in case the BFS sends a begin frame
   // before the missed frame task runs.
   missed_begin_frame_task_.Cancel();
 
   base::TimeTicks now = Now();
 
   // Discard missed begin frames if they are too late.
   if (adjusted_args.type == BeginFrameArgs::MISSED &&
       now > adjusted_args.deadline) {
-    begin_frame_source_->DidFinishFrame(this, 0);
+    // TODO(eseckler): Determine and set correct |ack.latest_confirmed_frame|.
+    BeginFrameAck ack(adjusted_args.source_id, adjusted_args.sequence_number,
+                      adjusted_args.sequence_number, 0, false);
+    begin_frame_source_->DidFinishFrame(this, ack);
     return;
   }
 
   // Run the previous deadline if any.
   if (state_machine_.begin_impl_frame_state() ==
       SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_INSIDE_BEGIN_FRAME) {
     OnBeginImplFrameDeadline();
     // We may not need begin frames any longer.
     if (!observing_begin_frame_source_) {
-      begin_frame_source_->DidFinishFrame(this, 0);
+      // TODO(eseckler): Determine and set correct |ack.latest_confirmed_frame|.
+      BeginFrameAck ack(adjusted_args.source_id, adjusted_args.sequence_number,
+                        adjusted_args.sequence_number, 0, false);
+      begin_frame_source_->DidFinishFrame(this, ack);
       return;
     }
   }
   DCHECK_EQ(state_machine_.begin_impl_frame_state(),
             SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_IDLE);
 
   bool main_thread_is_in_high_latency_mode =
       state_machine_.main_thread_missed_last_deadline();
   TRACE_EVENT2("cc,benchmark", "Scheduler::BeginImplFrame", "args",
                adjusted_args.AsValue(), "main_thread_missed_last_deadline",
@@ skipping 38 matching lines @@
                                                  bmf_to_activate_estimate, now);
 
   if (ShouldRecoverMainLatency(adjusted_args, can_activate_before_deadline)) {
     TRACE_EVENT_INSTANT0("cc", "SkipBeginMainFrameToReduceLatency",
                          TRACE_EVENT_SCOPE_THREAD);
     state_machine_.SetSkipNextBeginMainFrameToReduceLatency();
   } else if (ShouldRecoverImplLatency(adjusted_args,
                                       can_activate_before_deadline)) {
     TRACE_EVENT_INSTANT0("cc", "SkipBeginImplFrameToReduceLatency",
                          TRACE_EVENT_SCOPE_THREAD);
-    if (begin_frame_source_)
-      begin_frame_source_->DidFinishFrame(this, 0);
+    if (begin_frame_source_) {
+      // TODO(eseckler): Determine and set correct |ack.latest_confirmed_frame|.
+      BeginFrameAck ack(adjusted_args.source_id, adjusted_args.sequence_number,
+                        adjusted_args.sequence_number, 0, false);
+      begin_frame_source_->DidFinishFrame(this, ack);
+    }
     return;
   }
 
   BeginImplFrame(adjusted_args);
 }
 
 void Scheduler::BeginImplFrameSynchronous(const BeginFrameArgs& args) {
   TRACE_EVENT1("cc,benchmark", "Scheduler::BeginImplFrame", "args",
                args.AsValue());
 
   // The main thread currently can't commit before we draw with the
   // synchronous compositor, so never consider the BeginMainFrame fast.
   state_machine_.SetCriticalBeginMainFrameToActivateIsFast(false);
   begin_main_frame_args_ = args;
   begin_main_frame_args_.on_critical_path = !ImplLatencyTakesPriority();
 
   BeginImplFrame(args);
   compositor_timing_history_->WillFinishImplFrame(
       state_machine_.needs_redraw());
   FinishImplFrame();
 }
 
 void Scheduler::FinishImplFrame() {
   state_machine_.OnBeginImplFrameIdle();
   ProcessScheduledActions();
 
   client_->DidFinishImplFrame();
-  if (begin_frame_source_)
-    begin_frame_source_->DidFinishFrame(this, 0);
+  if (begin_frame_source_) {
+    // TODO(eseckler): Determine and set correct |ack.latest_confirmed_frame|.
+    BeginFrameAck ack(begin_main_frame_args_.source_id,
+                      begin_main_frame_args_.sequence_number,
+                      begin_main_frame_args_.sequence_number, 0,
+                      state_machine_.did_submit_in_last_frame());
+    begin_frame_source_->DidFinishFrame(this, ack);
+  }
   begin_impl_frame_tracker_.Finish();
 }
 
 // BeginImplFrame starts a compositor frame that will wait up until a deadline
 // for a BeginMainFrame+activation to complete before it times out and draws
 // any asynchronous animation and scroll/pinch updates.
 void Scheduler::BeginImplFrame(const BeginFrameArgs& args) {
   DCHECK_EQ(state_machine_.begin_impl_frame_state(),
             SchedulerStateMachine::BEGIN_IMPL_FRAME_STATE_IDLE);
   DCHECK(begin_impl_frame_deadline_task_.IsCancelled());
@@ skipping 316 matching lines @@
 }
 
 bool Scheduler::IsBeginMainFrameSentOrStarted() const {
   return (state_machine_.begin_main_frame_state() ==
               SchedulerStateMachine::BEGIN_MAIN_FRAME_STATE_SENT ||
           state_machine_.begin_main_frame_state() ==
               SchedulerStateMachine::BEGIN_MAIN_FRAME_STATE_STARTED);
 }
 
 }  // namespace cc