Move frame_sink_id management to framesink_manager.cc/h from

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..c01345aa12c760ab84709b4a73cb31d71c5eae78 100644
--- a/components/display_compositor/gpu_compositor_frame_sink.cc
+++ b/components/display_compositor/gpu_compositor_frame_sink.cc
@@ -11,6 +11,7 @@ namespace display_compositor {
 GpuCompositorFrameSink::GpuCompositorFrameSink(
     GpuCompositorFrameSinkDelegate* delegate,
     cc::SurfaceManager* surface_manager,
+    cc::FrameSinkManager* framesink_manager,
     const cc::FrameSinkId& frame_sink_id,
     std::unique_ptr<cc::Display> display,
     std::unique_ptr<cc::BeginFrameSource> begin_frame_source,
@@ -20,10 +21,12 @@ GpuCompositorFrameSink::GpuCompositorFrameSink(
     : delegate_(delegate),
       support_(this,
                surface_manager,
+               framesink_manager,
                frame_sink_id,
                std::move(display),
                std::move(begin_frame_source)),
       surface_manager_(surface_manager),
+      framesink_manager_(framesink_manager),

[Fady Samuel, 2017/02/12 19:04:08]

What's the point of this member variable?

[k.devara, 2017/02/13 06:52:00]

You are right- the argument is just a pass-thru I don't need to store it in the
member variable. I will clean this up. 

       surface_tracker_(frame_sink_id),
       client_(std::move(client)),
       compositor_frame_sink_private_binding_(