Teach the native viewport service to draw a Surface into itself

mojo/services/native_viewport/native_viewport_impl.h

 // Copyright 2014 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 MOJO_SERVICES_NATIVE_VIEWPORT_IMPL_H_
 #define MOJO_SERVICES_NATIVE_VIEWPORT_IMPL_H_
 
 #include "base/memory/weak_ptr.h"
+#include "cc/surfaces/surface_id_allocator.h"
 #include "mojo/services/native_viewport/platform_viewport.h"
+#include "mojo/services/public/interfaces/gpu/gpu.mojom.h"
 #include "mojo/services/public/interfaces/native_viewport/native_viewport.mojom.h"
+#include "mojo/services/public/interfaces/surfaces/surfaces.mojom.h"
+#include "mojo/services/public/interfaces/surfaces/surfaces_service.mojom.h"
+#include "ui/gfx/geometry/rect.h"
 
 namespace ui {
 class Event;
 }
 
 namespace mojo {
 
 class NativeViewportImpl : public InterfaceImpl<NativeViewport>,
-                           public PlatformViewport::Delegate {
+                           public PlatformViewport::Delegate,
+                           public SurfaceClient {
  public:
-  NativeViewportImpl();
+  NativeViewportImpl(SurfacesServicePtr surfaces_service, GpuPtr gpu_service);
   virtual ~NativeViewportImpl();
 
   // InterfaceImpl<NativeViewport> implementation.
   virtual void Create(RectPtr bounds) OVERRIDE;
   virtual void Show() OVERRIDE;
   virtual void Hide() OVERRIDE;
   virtual void Close() OVERRIDE;
   virtual void SetBounds(RectPtr bounds) OVERRIDE;
+  virtual void SubmittedFrame(SurfaceIdPtr surface_id) OVERRIDE;
 
   // PlatformViewport::Delegate implementation.
   virtual void OnBoundsChanged(const gfx::Rect& bounds) OVERRIDE;
   virtual void OnAcceleratedWidgetAvailable(
       gfx::AcceleratedWidget widget) OVERRIDE;
   virtual bool OnEvent(ui::Event* ui_event) OVERRIDE;
   virtual void OnDestroyed() OVERRIDE;
 
   void AckEvent();
 
  private:
+  // SurfaceClient implementation.
+  virtual void ReturnResources(Array<ReturnedResourcePtr> resources) OVERRIDE;
 
-  gfx::AcceleratedWidget widget_;
+  void OnSurfaceConnectionCreated(SurfacePtr surface, uint32_t id_namespace);
+  void CreateViewportBoundSurface();
+  void SubmitFrame();
+
   scoped_ptr<PlatformViewport> platform_viewport_;
+  SurfacesServicePtr surfaces_service_;

[jamesr, 2014/08/26 23:05:50]

might be worth splitting all this crap out into a helper class. wdyt?

[sky, 2014/08/26 23:16:32]

Sure.

+  SurfacePtr surface_;
+  GpuPtr gpu_service_;
+  gfx::Rect bounds_;
+  uint64_t widget_id_;
+  scoped_ptr<cc::SurfaceIdAllocator> surface_id_allocator_;
+  cc::SurfaceId surface_id_;
+  cc::SurfaceId child_surface_id_;
   bool waiting_for_event_ack_;
   base::WeakPtrFactory<NativeViewportImpl> weak_factory_;
 };
 
 }  // namespace mojo
 
 #endif  // MOJO_SERVICES_NATIVE_VIEWPORT_IMPL_H_