GpuCompositorFrameSink implements cc::mojom::DisplayPrivate

services/ui/ws/frame_generator.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "services/ui/ws/frame_generator.h"
 
 #include <utility>
 
 #include "base/containers/adapters.h"
 #include "cc/output/compositor_frame.h"
@@ skipping 25 matching lines @@
   // FrameGenerator during destruction of |compositor_frame_sink_|.
   weak_factory_.InvalidateWeakPtrs();
   compositor_frame_sink_.reset();
 }
 
 void FrameGenerator::OnAcceleratedWidgetAvailable(
     gfx::AcceleratedWidget widget) {
   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
   // BeginFrame if the window manager changes.
   compositor_frame_sink_->SetNeedsBeginFrame(true);
 }
 
 void FrameGenerator::OnSurfaceCreated(const cc::SurfaceId& surface_id,
                                       ServerWindow* window) {
   DCHECK(surface_id.is_valid());
@@ skipping 284 matching lines @@
   cc::SurfaceId underlay_surface_id =
       window->compositor_frame_sink_manager()->GetLatestSurfaceId(
           mojom::CompositorFrameSinkType::UNDERLAY);
   if (underlay_surface_id.is_valid())
     RemoveFrameSinkReference(underlay_surface_id.frame_sink_id());
 }
 
 }  // namespace ws
 
 }  // namespace ui