DirectCompositorFrameSink Uses CompositorFrameSinkSupport

components/display_compositor/gpu_compositor_frame_sink.cc

Index: components/display_compositor/gpu_compositor_frame_sink.cc
diff --git a/components/display_compositor/gpu_compositor_frame_sink.cc b/components/display_compositor/gpu_compositor_frame_sink.cc
index 9a6347f146602e13ddc0e7a36ba5759fb34f825f..dfcb05b8a480bbf4d00d7e355ed03120fdd17915 100644
--- a/components/display_compositor/gpu_compositor_frame_sink.cc
+++ b/components/display_compositor/gpu_compositor_frame_sink.cc
@@ -18,12 +18,11 @@ GpuCompositorFrameSink::GpuCompositorFrameSink(
         compositor_frame_sink_private_request,
     cc::mojom::MojoCompositorFrameSinkClientPtr client)
     : delegate_(delegate),
-      support_(this,
-               surface_manager,
-               frame_sink_id,
-               std::move(display),
-               std::move(begin_frame_source)),
+      support_(this, surface_manager, frame_sink_id, display.get()),
       surface_manager_(surface_manager),
+      display_(std::move(display)),

[Fady Samuel, 2017/01/29 15:43:01]

Move to GpuDisplayCompositorFrameSInk

[Alex Z., 2017/01/30 20:37:49]

Done.

+      display_begin_frame_source_(std::move(begin_frame_source)),

[Fady Samuel, 2017/01/29 15:43:01]

Move to GpuDisplayCompositorFrameSInk

[Alex Z., 2017/01/30 20:37:49]

Done.

+
       surface_tracker_(frame_sink_id),
       client_(std::move(client)),
       compositor_frame_sink_private_binding_(
@@ -32,6 +31,9 @@ GpuCompositorFrameSink::GpuCompositorFrameSink(
   compositor_frame_sink_private_binding_.set_connection_error_handler(
       base::Bind(&GpuCompositorFrameSink::OnPrivateConnectionLost,
                  base::Unretained(this)));
+
+  if (display_.get())
+    display_->SetVisible(true);

[Fady Samuel, 2017/01/29 15:43:01]

Move to GpuDisplayCompositorFrameSink. No need to check if we have a display
there.

[Alex Z., 2017/01/30 20:37:49]

Done.

 }
 
 GpuCompositorFrameSink::~GpuCompositorFrameSink() {