DirectCompositorFrameSink Uses CompositorFrameSinkSupport

components/display_compositor/gpu_display_compositor_frame_sink.cc

Index: components/display_compositor/gpu_display_compositor_frame_sink.cc
diff --git a/components/display_compositor/gpu_display_compositor_frame_sink.cc b/components/display_compositor/gpu_display_compositor_frame_sink.cc
index 892409ae369cd840595326f16913f18883547427..fa37354e29ed496a225f8df3740899db312b88c4 100644
--- a/components/display_compositor/gpu_display_compositor_frame_sink.cc
+++ b/components/display_compositor/gpu_display_compositor_frame_sink.cc
@@ -20,15 +20,17 @@ GpuDisplayCompositorFrameSink::GpuDisplayCompositorFrameSink(
     : GpuCompositorFrameSink(delegate,
                              surface_manager,
                              frame_sink_id,
-                             std::move(display),
-                             std::move(begin_frame_source),
+                             display.get(),
                              std::move(compositor_frame_sink_private_request),
                              std::move(client)),
       binding_(this, std::move(request)),
-      display_private_binding_(this, std::move(display_private_request)) {
+      display_private_binding_(this, std::move(display_private_request)),
+      display_(std::move(display)),
+      display_begin_frame_source_(std::move(begin_frame_source)) {
   binding_.set_connection_error_handler(
       base::Bind(&GpuDisplayCompositorFrameSink::OnClientConnectionLost,
                  base::Unretained(this)));
+  display_->SetVisible(true);

[danakj, 2017/01/31 20:19:17]

Why is this?

[Alex Z., 2017/01/31 21:09:30]

DirectCompositorFrameSink doesn't call SetVisible so I am moving it 
here from CompositorFrameSinkSupport.

 }
 
 GpuDisplayCompositorFrameSink::~GpuDisplayCompositorFrameSink() = default;