Revamp OffscreenCanvas commit flow

third_party/WebKit/public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom

Index: third_party/WebKit/public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom
diff --git a/third_party/WebKit/public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom b/third_party/WebKit/public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom
index f4e4448d561060e6fd9d53c0f2bae50a8f2d5c4e..22766ea2cb1185ead668c0dc9d0065df65f27d75 100644
--- a/third_party/WebKit/public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom
+++ b/third_party/WebKit/public/platform/modules/offscreencanvas/offscreen_canvas_surface.mojom
@@ -5,22 +5,32 @@
 module blink.mojom;
 
 import "cc/ipc/compositor_frame.mojom";
+import "cc/ipc/frame_sink_id.mojom";
 import "cc/ipc/mojo_compositor_frame_sink.mojom";
 import "cc/ipc/surface_id.mojom";
 import "cc/ipc/surface_sequence.mojom";
 
 interface OffscreenCanvasSurface {
-  // TODO(619138): Make creation of surface id Async after implementation
-  // change on surface_id generator that makes client side be able to generate
-  // unique surface_id.id_namespace alone.
-  [Sync]
-  GetSurfaceId() => (cc.mojom.SurfaceId surface_id);
-
   Require(cc.mojom.SurfaceId surface_id, cc.mojom.SurfaceSequence sequence);
   Satisfy(cc.mojom.SurfaceSequence sequence);
 };
 
+interface OffscreenCanvasSurfaceClient {
+  // TODO(fsamuel, xlai): Replace this with DisplayCompositorClient
+  OnSurfaceCreated(cc.mojom.SurfaceId surface_id,
+                   int32 width,
+                   int32 height,
+                   float device_scale_factor);
+};
+
+interface OffscreenCanvasSurfaceFactory {
+  CreateOffscreenCanvasSurface(cc.mojom.FrameSinkId frame_sink_id,
+                               OffscreenCanvasSurfaceClient client,
+                               OffscreenCanvasSurface& service);
+};
+
 interface OffscreenCanvasCompositorFrameSinkProvider {
+  // TODO(fsamuel, xlai): Replace this with DisplayCompositor
   CreateCompositorFrameSink(cc.mojom.SurfaceId surface_id,

[Fady Samuel, 2016/12/15 22:28:12]

This should only take in a FrameSinkId

[xlai (Olivia), 2016/12/15 22:46:28]

Done.

                             cc.mojom.MojoCompositorFrameSinkClient client,
                             cc.mojom.MojoCompositorFrameSink& sink);