Implement aura::WindowPortMus::CreateCompositorFrameSink()

ui/aura/window.cc

 // Copyright (c) 2012 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 "ui/aura/window.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <utility>
@@ skipping 974 matching lines @@
       ui::GestureRecognizer::Get()->CleanupStateForConsumer(this);
   for (Window::Windows::iterator iter = children_.begin();
        iter != children_.end();
        ++iter) {
     state_modified |= (*iter)->CleanupGestureState();
   }
   return state_modified;
 }
 
 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 {
   return port_->GetSurfaceId();
 }
 
 void Window::OnPaintLayer(const ui::PaintContext& context) {
   Paint(context);
 }
 
@@ skipping 77 matching lines @@
     layer_name = "Unnamed Window";
 
   if (id_ != -1)
     layer_name += " " + base::IntToString(id_);
 
   layer()->set_name(layer_name);
 #endif
 }
 
 }  // namespace aura