Implement CompositorFrame Struct Traits

components/mus/public/cpp/window_surface.h

 // Copyright 2015 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.
 
 #ifndef COMPONENTS_MUS_PUBLIC_CPP_WINDOW_SURFACE_H_
 #define COMPONENTS_MUS_PUBLIC_CPP_WINDOW_SURFACE_H_
 
 #include <memory>
 
 #include "base/callback.h"
@@ skipping 17 matching lines @@
   // static
   static std::unique_ptr<WindowSurface> Create(
       std::unique_ptr<WindowSurfaceBinding>* surface_binding);
 
   ~WindowSurface() override;
 
   // Called to indicate that the current thread has assumed control of this
   // object.
   void BindToThread();
 
-  void SubmitCompositorFrame(cc::mojom::CompositorFramePtr frame,
+  void SubmitCompositorFrame(std::unique_ptr<cc::CompositorFrame> frame,

[rjkroege, 2016/06/23 19:13:55]

my preference would be to not have client code unless absolutely necessary
because it makes things more tedious for use from other languages. Are you
adverse to removing the client lib eventually?

[Fady Samuel, 2016/06/25 04:31:12]

This is a fairly thin wrapper around mojo interfaces. I didn't intend to remove
this any time soon but we can discuss further offline.

                              const base::Closure& callback);
 
   void set_client(WindowSurfaceClient* client) { client_ = client; }
 
  private:
   friend class Window;
 
   WindowSurface(mojo::InterfacePtrInfo<mojom::Surface> surface_info,
                 mojo::InterfaceRequest<mojom::SurfaceClient> client_request);
 
@@ skipping 27 matching lines @@
 
   mojo::InterfaceRequest<mojom::Surface> surface_request_;
   mojo::InterfacePtrInfo<mojom::SurfaceClient> surface_client_;
 
   DISALLOW_COPY_AND_ASSIGN(WindowSurfaceBinding);
 };
 
 }  // namespace mus
 
 #endif  // COMPONENTS_MUS_PUBLIC_CPP_WINDOW_SURFACE_H_