GpuCompositorFrameSink implements cc::mojom::DisplayPrivate

services/ui/ws/frame_generator.cc

Index: services/ui/ws/frame_generator.cc
diff --git a/services/ui/ws/frame_generator.cc b/services/ui/ws/frame_generator.cc
index acc12aff03803d6a58a963448b5638a8ec084fa2..8075cf2244e36bb1ab77fc8a9d5ca6ca7e938868 100644
--- a/services/ui/ws/frame_generator.cc
+++ b/services/ui/ws/frame_generator.cc
@@ -43,8 +43,12 @@ void FrameGenerator::OnAcceleratedWidgetAvailable(
   DCHECK_NE(gfx::kNullAcceleratedWidget, widget);
   cc::mojom::MojoCompositorFrameSinkRequest request =
       mojo::GetProxy(&compositor_frame_sink_);
+  cc::mojom::DisplayPrivatePtr display_private;

[Fady Samuel, 2016/12/16 05:29:39]

I think FrameGenerator probably wants to hold onto display_private in this case.
+kylechar@, +sadrul@

[Alex Z., 2016/12/16 16:42:00]

Done.

+  cc::mojom::DisplayPrivateRequest display_request =
+      mojo::GetProxy(&display_private);
   root_window_->CreateDisplayCompositorFrameSink(
-      widget, std::move(request), binding_.CreateInterfacePtrAndBind());
+      widget, std::move(request), binding_.CreateInterfacePtrAndBind(),
+      std::move(display_private), std::move(display_request));
   // TODO(fsamuel): This means we're always requesting a new BeginFrame signal
   // even when we don't need it. Once surface ID propagation work is done,
   // this will not be necessary because FrameGenerator will only need a