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..3bfede38479c00c9024ccd10dada89f5419e308a 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,21 +5,26 @@
 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);
+  SetClient(OffscreenCanvasSurfaceClient client,

[Fady Samuel, 2016/12/15 19:54:52]

I think we prefer not to have a SetClient interface but rather have a Factory
interface: OffscreenCanvasSurfaceFactory, that takes in both
OffscreenCanvasSurface and OffscreenCanvasSurfaceClient.

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

Done.

+            cc.mojom.FrameSinkId frame_sink_id);
 
   Require(cc.mojom.SurfaceId surface_id, cc.mojom.SurfaceSequence sequence);
   Satisfy(cc.mojom.SurfaceSequence sequence);
 };
 
+interface OffscreenCanvasSurfaceClient {

[Fady Samuel, 2016/12/15 19:54:52]

nit: Add TODO(fsamuel, xlai): Replace this with DisplayCompositorClient

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

Done.

+  OnSurfaceCreated(cc.mojom.SurfaceId surface_id,
+                   int32 width,
+                   int32 height,
+                   float device_scale_factor);
+};
+
 interface OffscreenCanvasCompositorFrameSinkProvider {

[Fady Samuel, 2016/12/15 19:54:52]

nit: Add TODO(fsamuel, xlai): Replace this with DisplayCompositor

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

Done.

   CreateCompositorFrameSink(cc.mojom.SurfaceId surface_id,
                             cc.mojom.MojoCompositorFrameSinkClient client,