[cc] Remove BeginFrameAck from BFS::DidFinishFrame and update tests.

cc/surfaces/display_scheduler.cc

 // Copyright 2015 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/surfaces/display_scheduler.h"
 
 #include <vector>
 
 #include "base/auto_reset.h"
 #include "base/stl_util.h"
@@ skipping 346 matching lines @@
 
 void DisplayScheduler::OnBeginFrameDeadline() {
   TRACE_EVENT0("cc", "DisplayScheduler::OnBeginFrameDeadline");
   DCHECK(inside_begin_frame_deadline_interval_);
 
   bool did_draw = AttemptDrawAndSwap();
   DidFinishFrame(did_draw);
 }
 
 void DisplayScheduler::DidFinishFrame(bool did_draw) {
-  // TODO(eseckler): Determine and set correct |ack.latest_confirmed_frame|.
-  BeginFrameAck ack(current_begin_frame_args_.source_id,
-                    current_begin_frame_args_.sequence_number,
-                    current_begin_frame_args_.sequence_number, did_draw);
-  begin_frame_source_->DidFinishFrame(this, ack);
+  // TODO(eseckler): Let client know that frame was completed.

[sunnyps, 2017/05/24 02:38:26]

nit: DCHECK(begin_frame_source_) here

[Eric Seckler, 2017/05/24 09:00:40]

Done.

+  begin_frame_source_->DidFinishFrame(this);
 }
 
 void DisplayScheduler::DidSwapBuffers() {
   pending_swaps_++;
   uint32_t swap_id = next_swap_id_++;
   TRACE_EVENT_ASYNC_BEGIN0("cc", "DisplayScheduler:pending_swaps", swap_id);
 }
 
 void DisplayScheduler::DidReceiveSwapBuffersAck() {
   uint32_t swap_id = next_swap_id_ - pending_swaps_;
   pending_swaps_--;
   TRACE_EVENT_ASYNC_END0("cc", "DisplayScheduler:pending_swaps", swap_id);
   ScheduleBeginFrameDeadline();
 }
 
 }  // namespace cc