Passing FrameSinkId to WindowCompositorFrameSink

ui/aura/mus/window_port_mus.cc

Index: ui/aura/mus/window_port_mus.cc
diff --git a/ui/aura/mus/window_port_mus.cc b/ui/aura/mus/window_port_mus.cc
index baba5ca7f1819823f2f9ec8f2b88db71b713d897..b4f638319a60ef36f28043954e6378cb24d2fca0 100644
--- a/ui/aura/mus/window_port_mus.cc
+++ b/ui/aura/mus/window_port_mus.cc
@@ -79,10 +79,11 @@ WindowPortMus::RequestCompositorFrameSink(
     gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager) {
   std::unique_ptr<WindowCompositorFrameSinkBinding>
       compositor_frame_sink_binding;
+  cc::FrameSinkId frame_sink_id(server_id(), 0);

[Fady Samuel, 2017/01/13 19:39:05]

No point in having a member variable

[Saman Sami, 2017/01/13 19:41:52]

Done.

   std::unique_ptr<WindowCompositorFrameSink> compositor_frame_sink =
-      WindowCompositorFrameSink::Create(std::move(context_provider),
-                                        gpu_memory_buffer_manager,
-                                        &compositor_frame_sink_binding);
+      WindowCompositorFrameSink::Create(
+          frame_sink_id, std::move(context_provider), gpu_memory_buffer_manager,

[Fady Samuel, 2017/01/13 19:39:05]

cc::FrameSinkId(server_id(), 0)

[Saman Sami, 2017/01/13 19:41:52]

Done.

+          &compositor_frame_sink_binding);
   AttachCompositorFrameSink(std::move(compositor_frame_sink_binding));
   return compositor_frame_sink;
 }