Implement aura::Window::CreateCompositorFrameSink()

cc/output/compositor_frame_sink.h

Index: cc/output/compositor_frame_sink.h
diff --git a/cc/output/compositor_frame_sink.h b/cc/output/compositor_frame_sink.h
index f8b7a47b4e6c97fc06934a4a4062377f270e1e45..dfe5fa17010dbf5a6cfaee75fbf9800382b1b9a9 100644
--- a/cc/output/compositor_frame_sink.h
+++ b/cc/output/compositor_frame_sink.h
@@ -16,6 +16,7 @@
 #include "cc/output/overlay_candidate_validator.h"
 #include "cc/output/vulkan_context_provider.h"
 #include "cc/resources/returned_resource.h"
+#include "cc/surfaces/surface_id.h"
 #include "gpu/command_buffer/common/texture_in_use_response.h"
 #include "ui/gfx/color_space.h"
 
@@ -121,6 +122,13 @@ class CC_EXPORT CompositorFrameSink {
   // processed in order to unthrottle the next frame.
   virtual void SubmitCompositorFrame(CompositorFrame frame) = 0;
 
+  // Notify that the surface is no longer in use (and is okay to be evicted) so
+  // that its resources gets returned in time.
+  virtual void EvictFrame() {}

[Fady Samuel, 2017/05/08 15:52:09]

Is this necessary? Can we avoid adding this?

[Peng, 2017/05/08 17:44:39]

The exo need call this function to flush frames before destroying the
CompositorFrameSink. And exo need keep CompositorFrameSink alive before all
submitted resources are released by ReclaimResources(). So flushing frame during
destruction or DetachFromCliend() doesn't work.

[Fady Samuel, 2017/05/08 18:17:35]

OK Thanks.

+
+  // Get the current surface id.
+  virtual SurfaceId GetSurfaceId() const;

[Fady Samuel, 2017/05/08 15:52:09]

Is this necessary, can we avoid adding this?

[Peng, 2017/05/08 17:44:39]

Done

+
  protected:
   // Bound to the ContextProvider to hear about when it is lost and inform the
   // |client_|.