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

Unified Diff: cc/ipc/mojo_compositor_frame_sink.mojom

Issue 2551083003: Add Satisfy() and Require() to MojoCompositorFrameSink (Closed)
Patch Set: 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
« no previous file with comments | « no previous file | content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..bd45ec4944c3d0d1efaa0669b5e5eb2793db688e 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 provided |sequence| as a destruction dependency of the
Fady Samuel 2016/12/05 20:03:09 Please add a TODO here to delete these once surfac
Alex Z. 2016/12/05 20:07:49 Done.
+ // surface associated with the provided |local_frame_id|.
+ 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.
};
« no previous file with comments | « no previous file | content/browser/renderer_host/offscreen_canvas_compositor_frame_sink.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698