DelegatedFrameHost should handle satisfied sequences in frame metadata.

content/browser/compositor/delegated_frame_host.h

 // Copyright 2014 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.
 
 #ifndef CONTENT_BROWSER_COMPOSITOR_DELEGATED_FRAME_HOST_H_
 #define CONTENT_BROWSER_COMPOSITOR_DELEGATED_FRAME_HOST_H_
 
 #include "cc/layers/delegated_frame_provider.h"
 #include "cc/layers/delegated_frame_resource_collection.h"
 #include "cc/output/copy_output_result.h"
@@ skipping 105 matching lines @@
   // cc::SurfaceFactoryClient implementation.
   void ReturnResources(const cc::ReturnedResourceArray& resources) override;
 
   bool CanCopyToBitmap() const;
 
   // Public interface exposed to RenderWidgetHostView.
   void SwapDelegatedFrame(
       uint32 output_surface_id,
       scoped_ptr<cc::DelegatedFrameData> frame_data,
       float frame_device_scale_factor,
-      const std::vector<ui::LatencyInfo>& latency_info);
+      const std::vector<ui::LatencyInfo>& latency_info,
+      std::vector<uint32_t> satifies_sequences);

[jochen (gone - plz use gerrit), 2015/07/14 14:33:00]

why is this pass by value? I'd either expect a const ref or a pointer

[wjmaclean, 2015/07/14 14:58:01]

My bad ... the function modifies the vector (via swap), so it should have been a
pointer. Since we generally have few, if any, sequences in the vector copying
isn't *totally* awful, but I think we want to be clear that SwapDelegatedFrame
is 'consuming' the sequences in this vector.

Fixed.

   void WasHidden();
   void WasShown(const ui::LatencyInfo& latency_info);
   void WasResized();
   bool HasSavedFrame();
   gfx::Size GetRequestedRendererSize() const;
   void SetCompositor(ui::Compositor* compositor);
   void ResetCompositor();
   void SetVSyncParameters(const base::TimeTicks& timebase,
                           const base::TimeDelta& interval);
   // Note: |src_subset| is specified in DIP dimensions while |output_size|
@@ skipping 192 matching lines @@
   // YUV readback pipeline.
   scoped_ptr<content::ReadbackYUVInterface>
       yuv_readback_pipeline_;
 
   scoped_ptr<DelegatedFrameEvictor> delegated_frame_evictor_;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_COMPOSITOR_DELEGATED_FRAME_HOST_H_