Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1013)

Unified Diff: cc/ipc/mojo_compositor_frame_sink.mojom

Issue 2493223002: Change exo::SurfaceFactoryOwner to exo::ExoCompositorFrameSink (Closed)
Patch Set: Added MojoCompositorFrameSink::Require() and MojoCompositorFrameSink::Satisfy() Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/ipc/mojo_compositor_frame_sink.mojom
diff --git a/cc/ipc/mojo_compositor_frame_sink.mojom b/cc/ipc/mojo_compositor_frame_sink.mojom
index 5c6d77e38e494c907a98115ce8aa65301b3db908..20a3e415a1450cf5c146c21f84c7e0f6346f110a 100644
--- a/cc/ipc/mojo_compositor_frame_sink.mojom
+++ b/cc/ipc/mojo_compositor_frame_sink.mojom
@@ -9,6 +9,7 @@ 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.
@@ -35,6 +36,14 @@ interface MojoCompositorFrameSink {
// 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 destr
Alex Z. 2016/12/06 20:10:41 Done. The change has also been made to the separat
+ 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.
};

Powered by Google App Engine
This is Rietveld 408576698