GpuDisplayCompositorFrameSink => GpuRootCompositorFrameSink

cc/ipc/display_compositor.mojom

Index: cc/ipc/display_compositor.mojom
diff --git a/cc/ipc/display_compositor.mojom b/cc/ipc/display_compositor.mojom
index bd0e942844edfe09cbbf536c53f06dcdc64c16a4..579e054454d0706201012d0f0d3b8b0a62f65447 100644
--- a/cc/ipc/display_compositor.mojom
+++ b/cc/ipc/display_compositor.mojom
@@ -40,8 +40,8 @@ interface DisplayPrivate {
 interface DisplayCompositor {
   // Create a CompositorFrameSink for a privileged client (e.g. WindowServer).
   // This is only used by privileged clients. The client can call methods that
-  // talks to the cc::Display (e.g. ResizeDisplay(), SetDisplayVisible(), etc)
-  CreateDisplayCompositorFrameSink(
+  // talks to the Display (e.g. ResizeDisplay(), SetDisplayVisible(), etc)
+  CreateRootCompositorFrameSink(
       cc.mojom.FrameSinkId frame_sink_id,
       gpu.mojom.SurfaceHandle widget,
       associated cc.mojom.MojoCompositorFrameSink& compositor_frame_sink,
@@ -49,10 +49,10 @@ interface DisplayCompositor {
       cc.mojom.MojoCompositorFrameSinkClient compositor_frame_sink_client,
       associated DisplayPrivate& display_private);
 
-  // CreateOffscreenCompositorFrameSink is used by unprivileged clients.
-  // An off screen CompositorFrameSink only submits frames to a buffer instead
+  // CreateCompositorFrameSink is used by unprivileged clients.
+  // This CompositorFrameSink only submits frames to a buffer instead

[danakj, 2017/02/23 22:13:44]

This comment seems like it needs fixing, saying its to a buffer instead of a
display? It's not a root and has to be parented by something to appear, rather?

[Fady Samuel, 2017/02/23 23:14:58]

Done.

   // of a display.
-  CreateOffscreenCompositorFrameSink(
+  CreateCompositorFrameSink(
       cc.mojom.FrameSinkId frame_sink_id,
       cc.mojom.MojoCompositorFrameSink& compositor_frame_sink,
       cc.mojom.MojoCompositorFrameSinkPrivate& compositor_frame_sink_private,