Change exo::SurfaceFactoryOwner to exo::ExoCompositorFrameSink

cc/ipc/mojo_compositor_frame_sink.mojom

 // 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.
 
 module cc.mojom;
 
 import "cc/ipc/begin_frame_args.mojom";
 import "cc/ipc/compositor_frame.mojom";
 import "cc/ipc/frame_sink_id.mojom";
 import "cc/ipc/local_frame_id.mojom";
 import "cc/ipc/returned_resource.mojom";
+import "cc/ipc/surface_sequence.mojom";
 
 // A MojoCompositorFrameSink is an interface for receiving CompositorFrame
 // structs. A CompositorFrame contains the complete output meant for display.
 // Each time a client has a graphical update, and receives an OnBeginFrame, it
 // is responsible for creating a CompositorFrame to update its portion of the
 // screen.
 interface MojoCompositorFrameSink {
   // Lets the display compositor know that the client wishes to receive the next
   // BeginFrame event.
   SetNeedsBeginFrame(bool needs_begin_frame);
 
   // Submits a CompositorFrame to the display compositor that will be presented
   // to screen the next time frames from all CompositorFrameSinks are aggregated
   // to produce a display CompositorFrame. If a client wishes to allocate a new
   // surface (e.g. during resize), then it can simply allocate a new
   // |local_frame_id|.
   // For successful swaps, the implementation must call
   // DidReceiveCompositorFrameAck() asynchronously when the frame has been
   // processed in order to unthrottle the next frame.
   SubmitCompositorFrame(cc.mojom.LocalFrameId local_frame_id,
                         cc.mojom.CompositorFrame frame);
 
   // Notify that the surface is no longer in use (and is okay to be evicted) so
   // that its resources gets returned in time.
   EvictFrame();
 
+  // Add the surface associated with the id to the destruction dependency of the
+  // sequence.

[Fady Samuel, 2016/12/05 16:40:31]

How about: "Add the provided |sequence| as a destruction dependency of the
surface associated with the provided |local_frame_id|."

Also, Add a TODO(fsamuel, kylechar, staraz): Remove these methods once surface
references are ready.

[Alex Z., 2016/12/06 20:10:41]

Done. The change has also been made to the separate CL.

+  Require(cc.mojom.LocalFrameId local_frame_id,
+          cc.mojom.SurfaceSequence sequence);
+
+  // Mark the sequence as satisfied and garbage collect surfaces.
+  Satisfy(cc.mojom.SurfaceSequence sequence);
+
   // TODO(fsamuel): ReadbackBitmap API would be useful here.
 };
 
 interface MojoCompositorFrameSinkClient {
   // Notification that the previous CompositorFrame given to
   // SubmitCompositorFrame() has been processed and that another frame
   // can be submitted. This provides backpressure from the display compositor
   // so that frames are submitted only at the rate it can handle them.
   // TODO(fsamuel): This method ought not be necessary with unified BeginFrame.
   // However, there's a fair amount of cleanup and refactoring necessary to get
@@ skipping 11 matching lines @@
 };
 
 // MojoCompositorFrameSinkPrivate is used by the display compositor host to set
 // up BeginFrameSource hierarchies.  This API lives in SurfaceManager but is
 // called from here in order to ensure that hierarchy registration does not race
 // CompositorFrameSink creation.
 interface MojoCompositorFrameSinkPrivate {
   AddChildFrameSink(FrameSinkId child_frame_sink_id);
   RemoveChildFrameSink(FrameSinkId child_frame_sink_id);
 };