Implement aura::WindowPortMus::CreateCompositorFrameSink()

ui/aura/window.cc

Index: ui/aura/window.cc
diff --git a/ui/aura/window.cc b/ui/aura/window.cc
index 44bb18bef197ec96fdd37a771482b9f233c351f0..893af1cd3f3ac8d442bb66dffc805af6789e6c88 100644
--- a/ui/aura/window.cc
+++ b/ui/aura/window.cc
@@ -992,7 +992,9 @@ bool Window::CleanupGestureState() {
 }
 
 std::unique_ptr<cc::CompositorFrameSink> Window::CreateCompositorFrameSink() {
-  return port_->CreateCompositorFrameSink();
+  return port_->CreateCompositorFrameSink(
+      nullptr,

[sadrul, 2017/05/26 20:22:25]

these extra params don't seem necessary? Can WindowPortMus just get the
gmb-manager from aura::Env itself? (and just remove the context_provider param
since it's always null?)

[Peng, 2017/05/26 20:41:32]

I added this two args, because we want to get rid of
WindowPortMus::RequestCompositorFrameSink(). See [1] for detail.

[1]
https://codereview.chromium.org/2875753002/diff2/220001:240001/ui/aura/mus/wi...

[sadrul, 2017/05/27 01:43:22]

I see. I guess you need the ContextProvider param, but you still should not need
the gmb-manager param. (i.e. using Env::context_factory() to get the gmb-manager
should still work just OK)

+      aura::Env::GetInstance()->context_factory()->GetGpuMemoryBufferManager());
 }
 
 cc::SurfaceId Window::GetSurfaceId() const {