[ui/ws] Acknowledge BeginFrames in FrameGenerator + add tests.

services/ui/ws/frame_generator.cc

 // Copyright 2016 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 "services/ui/ws/frame_generator.h"
 
 #include <utility>
 #include <vector>
 
 #include "cc/output/compositor_frame.h"
@@ skipping 80 matching lines @@
                             const gfx::Rect& viewport,
                             bool resourceless_software_draw) {}
 
 void FrameGenerator::SetMemoryPolicy(const cc::ManagedMemoryPolicy& policy) {}
 
 void FrameGenerator::SetExternalTilePriorityConstraints(
     const gfx::Rect& viewport_rect,
     const gfx::Transform& transform) {}
 
 void FrameGenerator::OnBeginFrame(const cc::BeginFrameArgs& begin_frame_args) {
+  current_begin_frame_ack_ = cc::BeginFrameAck(
+      begin_frame_args.source_id, begin_frame_args.sequence_number,
+      begin_frame_args.sequence_number, 0, false);
   if (!root_window_->visible() ||
       begin_frame_args.type == cc::BeginFrameArgs::MISSED) {
+    begin_frame_source_->DidFinishFrame(this, current_begin_frame_ack_);
     return;
   }
 
+  current_begin_frame_ack_.has_damage = true;
+  last_begin_frame_args_ = begin_frame_args;
+
   // TODO(fsamuel): We should add a trace for generating a top level frame.
   cc::CompositorFrame frame(GenerateCompositorFrame(root_window_->bounds()));
+  compositor_frame_sink_->SubmitCompositorFrame(std::move(frame));
 
-  compositor_frame_sink_->SubmitCompositorFrame(std::move(frame));
+  begin_frame_source_->DidFinishFrame(this, current_begin_frame_ack_);
   SetNeedsBeginFrame(false);
-  last_begin_frame_args_ = begin_frame_args;
 }
 
 const cc::BeginFrameArgs& FrameGenerator::LastUsedBeginFrameArgs() const {
   return last_begin_frame_args_;
 }
 
 void FrameGenerator::OnBeginFrameSourcePausedChanged(bool paused) {}
 
 cc::CompositorFrame FrameGenerator::GenerateCompositorFrame(
     const gfx::Rect& output_rect) {
@@ skipping 21 matching lines @@
         cc::FilterOperation::CreateInvertFilter(1.f));
     quad->SetNew(shared_state, output_rect, output_rect, render_pass_id,
                  0 /* mask_resource_id */, gfx::RectF() /* mask_uv_rect */,
                  gfx::Size() /* mask_texture_size */,
                  gfx::Vector2dF() /* filters_scale */,
                  gfx::PointF() /* filters_origin */,
                  gfx::RectF() /* tex_coord_rect */);
     frame.render_pass_list.push_back(std::move(invert_pass));
   }
   frame.metadata.device_scale_factor = device_scale_factor_;
+  frame.metadata.begin_frame_ack = current_begin_frame_ack_;
 
   if (window_manager_surface_info_.is_valid()) {
     frame.metadata.referenced_surfaces.push_back(
         window_manager_surface_info_.id());
   }
 
   return frame;
 }
 
 void FrameGenerator::DrawWindow(cc::RenderPass* pass) {
@@ skipping 34 matching lines @@
   observing_begin_frames_ = needs_begin_frame;
   if (needs_begin_frame)
     begin_frame_source_->AddObserver(this);
   else
     begin_frame_source_->RemoveObserver(this);
 }
 
 }  // namespace ws
 
 }  // namespace ui