Implement aura::WindowPortMus::CreateCompositorFrameSink()

components/exo/surface.cc

Index: components/exo/surface.cc
diff --git a/components/exo/surface.cc b/components/exo/surface.cc
index 301b8d96cdf52b2cbd112b2769a41e2383775d1d..f5d3b2f84d255bc727a57bd5f115afca9b9c4aff 100644
--- a/components/exo/surface.cc
+++ b/components/exo/surface.cc
@@ -167,10 +167,6 @@ class CustomWindowTargeter : public aura::WindowTargeter {
 ////////////////////////////////////////////////////////////////////////////////
 // Surface, public:
 
-// TODO(fsamuel): exo should not use context_factory_private. Instead, we should
-// request a CompositorFrameSink from the aura::Window. Setting up the
-// BeginFrame hierarchy should be an internal implementation detail of aura or
-// mus in aura-mus.
 Surface::Surface() : window_(new aura::Window(new CustomWindowDelegate(this))) {
   window_->SetType(aura::client::WINDOW_TYPE_CONTROL);
   window_->SetName("ExoSurface");
@@ -790,6 +786,7 @@ void Surface::UpdateSurface(bool full_damage) {
   }
 
   content_size_ = layer_size;
+  window_->SetBounds(gfx::Rect(window_->bounds().origin(), content_size_));

[Fady Samuel, 2017/05/30 18:36:41]

Why?

[Peng, 2017/05/30 19:46:28]

ClientSurfaceEmbedder will read window's size and set it to an extra layer. See
[1]

For cash, we will set the frame's size to window_->layer() in WindowPortLocal.
And aura::Window will observe the window->layer()'s size changes and update
itself.

[1]
https://cs.chromium.org/chromium/src/ui/aura/mus/client_surface_embedder.cc?q...

   // TODO(jbauman): Figure out how this interacts with the pixel size of
   // CopyOutputRequests on the layer.
   gfx::Size contents_surface_size = layer_size;