Convert view manager to surfaces with uploading shim in client lib

mojo/services/native_viewport/native_viewport_impl.cc

 // 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.
 
 #include "mojo/services/native_viewport/native_viewport_impl.h"
 
 #include "base/bind.h"
 #include "base/macros.h"
 #include "base/message_loop/message_loop.h"
 #include "base/time/time.h"
@@ skipping 23 matching lines @@
   // TODO(jamesr): Should be mojo_gpu_service
   app->ConnectToService("mojo:mojo_native_viewport_service", &gpu_service_);
 }
 
 NativeViewportImpl::~NativeViewportImpl() {
   // Destroy the NativeViewport early on as it may call us back during
   // destruction and we want to be in a known state.
   platform_viewport_.reset();
 }
 
-void NativeViewportImpl::Create(RectPtr bounds) {
+void NativeViewportImpl::Create(SizePtr bounds) {
   platform_viewport_ = PlatformViewport::Create(this);
-  platform_viewport_->Init(bounds.To<gfx::Rect>());
-  OnBoundsChanged(bounds.To<gfx::Rect>());
+  gfx::Rect rect = gfx::Rect(bounds.To<gfx::Size>());
+  platform_viewport_->Init(rect);
+  OnBoundsChanged(rect);
 }
 
 void NativeViewportImpl::Show() {
   platform_viewport_->Show();
 }
 
 void NativeViewportImpl::Hide() {
   platform_viewport_->Hide();
 }
 
 void NativeViewportImpl::Close() {
   DCHECK(platform_viewport_);
   platform_viewport_->Close();
 }
 
-void NativeViewportImpl::SetBounds(RectPtr bounds) {
-  platform_viewport_->SetBounds(bounds.To<gfx::Rect>());
+void NativeViewportImpl::SetBounds(SizePtr bounds) {
+  platform_viewport_->SetBounds(gfx::Rect(bounds.To<gfx::Size>()));
 }
 
 void NativeViewportImpl::SubmittedFrame(SurfaceIdPtr child_surface_id) {
   if (child_surface_id_.is_null()) {
     // If this is the first indication that the client will use surfaces,
     // initialize that system.
     // TODO(jamesr): When everything is converted to surfaces initialize this
     // eagerly.
     viewport_surface_.reset(
         new ViewportSurface(surfaces_service_.get(),
                             gpu_service_.get(),
-                            bounds_,
+                            bounds_.size(),
                             child_surface_id.To<cc::SurfaceId>()));
     if (widget_id_)
       viewport_surface_->SetWidgetId(widget_id_);
   }
   child_surface_id_ = child_surface_id.To<cc::SurfaceId>();
   if (viewport_surface_)
     viewport_surface_->SetChildId(child_surface_id_);
 }
 
 void NativeViewportImpl::OnBoundsChanged(const gfx::Rect& bounds) {
   bounds_ = bounds;
-  client()->OnBoundsChanged(Rect::From(bounds));
+  client()->OnBoundsChanged(Size::From(bounds.size()));
   if (viewport_surface_)
-    viewport_surface_->SetBounds(bounds);
+    viewport_surface_->SetSize(bounds.size());
 }
 
 void NativeViewportImpl::OnAcceleratedWidgetAvailable(
     gfx::AcceleratedWidget widget) {
   widget_id_ = static_cast<uint64_t>(bit_cast<uintptr_t>(widget));
   // TODO(jamesr): Remove once everything is converted to surfaces.
   client()->OnCreated(widget_id_);
   if (viewport_surface_)
     viewport_surface_->SetWidgetId(widget_id_);
 }
@@ skipping 26 matching lines @@
 void NativeViewportImpl::OnDestroyed() {
   client()->OnDestroyed();
 }
 
 void NativeViewportImpl::AckEvent() {
   waiting_for_event_ack_ = false;
 }
 
 }  // namespace mojo