Fixes for surfaces bindings exposed by wm_flow app

mojo/services/public/cpp/view_manager/lib/view_manager_client_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/public/cpp/view_manager/lib/view_manager_client_impl.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop/message_loop.h"
 #include "base/stl_util.h"
@@ skipping 79 matching lines @@
         ViewPrivate(root_).view_manager())->RemoveRoot(root_);
     view->RemoveObserver(this);
     delete this;
   }
 
   View* root_;
 
   DISALLOW_COPY_AND_ASSIGN(RootObserver);
 };
 
-bool CreateMapAndDupSharedBuffer(size_t size,
-                                  void** memory,
-                                  ScopedSharedBufferHandle* handle,
-                                  ScopedSharedBufferHandle* duped) {
-  MojoResult result = CreateSharedBuffer(NULL, size, handle);
-  if (result != MOJO_RESULT_OK)
-    return false;
-  DCHECK(handle->is_valid());
-
-  result = DuplicateBuffer(handle->get(), NULL, duped);
-  if (result != MOJO_RESULT_OK)
-    return false;
-  DCHECK(duped->is_valid());
-
-  result = MapBuffer(
-      handle->get(), 0, size, memory, MOJO_MAP_BUFFER_FLAG_NONE);
-  if (result != MOJO_RESULT_OK)
-    return false;
-  DCHECK(*memory);
-
-  return true;
-}
-
 ViewManagerClientImpl::ViewManagerClientImpl(ViewManagerDelegate* delegate,
                                              Shell* shell)
     : connected_(false),
       connection_id_(0),
       next_id_(1),
       delegate_(delegate),
       window_manager_delegate_(NULL),
       shell_(shell) {
   // TODO(beng): Come up with a better way of establishing a configuration for
   //             what the active window manager is.
@@ skipping 19 matching lines @@
       non_owned.push_back(it->second);
       views_.erase(it);
     }
   }
   // Delete the non-owned views last. In the typical case these are roots. The
   // exception is the window manager, which may know aboutother random views
   // that it doesn't own.
   // NOTE: we manually delete as we're a friend.
   for (size_t i = 0; i < non_owned.size(); ++i)
     delete non_owned[i];
+
+  for (BitmapUploaderMap::iterator it = bitmap_uploaders_.begin();
+       it != bitmap_uploaders_.end();
+       ++it) {
+    delete it->second;

[Ben Goodger (Google), 2014/09/13 01:22:22]

rather than track this here, do you think it makes sense now to move the bitmap
uploader instance to the view class?

+  }
   delegate_->OnViewManagerDisconnected(this);
 }
 
 Id ViewManagerClientImpl::CreateView() {
   DCHECK(connected_);
   const Id view_id = MakeTransportId(connection_id_, ++next_id_);
   service_->CreateView(view_id, ActionCompletedCallbackWithErrorCode());
   return view_id;
 }
 
@@ skipping 35 matching lines @@
   DCHECK(connected_);
   if (surface_id.is_null())
     return;
   service_->SetViewSurfaceId(
       view_id, surface_id.Pass(), ActionCompletedCallback());
 }
 
 void ViewManagerClientImpl::SetViewContents(Id view_id,
                                             const SkBitmap& contents) {
   DCHECK(connected_);
-  if (!bitmap_uploader_) {
-    SurfacesServicePtr surfaces_service;
-    InterfacePtr<ServiceProvider> surfaces_service_provider;
-    shell_->ConnectToApplication("mojo:mojo_surfaces_service",
-                                 Get(&surfaces_service_provider));
-    ConnectToService(surfaces_service_provider.get(), &surfaces_service);
-    GpuPtr gpu_service;
-    InterfacePtr<ServiceProvider> gpu_service_provider;
-    shell_->ConnectToApplication("mojo:mojo_native_viewport_service",
-                                 Get(&gpu_service_provider));
-    ConnectToService(gpu_service_provider.get(), &gpu_service);
-    bitmap_uploader_.reset(
-        new BitmapUploader(surfaces_service.Pass(), gpu_service.Pass()));
-  }
-  bitmap_uploader_->Upload(
-      contents,
-      base::Bind(&ViewManagerClientImpl::SetSurfaceId,
-                 // We'll destroy the bitmap_uploader before we are destroyed,
-                 // so we can use an unretained pointer here.
-                 base::Unretained(this),
-                 view_id));
+  BitmapUploaderForView(view_id)->SetBitmap(contents);
+}
+
+void ViewManagerClientImpl::SetViewColor(Id view_id, SkColor color) {
+  DCHECK(connected_);
+  BitmapUploaderForView(view_id)->SetColor(color);
 }
 
 void ViewManagerClientImpl::SetFocus(Id view_id) {
   window_manager_->FocusWindow(view_id, ActionCompletedCallback());
 }
 
 void ViewManagerClientImpl::SetVisible(Id view_id, bool visible) {
   DCHECK(connected_);
   service_->SetViewVisibility(view_id, visible, ActionCompletedCallback());
 }
@@ skipping 217 matching lines @@
   return base::Bind(&ViewManagerClientImpl::OnActionCompleted,
                     base::Unretained(this));
 }
 
 base::Callback<void(ErrorCode)>
     ViewManagerClientImpl::ActionCompletedCallbackWithErrorCode() {
   return base::Bind(&ViewManagerClientImpl::OnActionCompletedWithErrorCode,
                     base::Unretained(this));
 }
 
+BitmapUploader* ViewManagerClientImpl::BitmapUploaderForView(Id view_id) {
+  BitmapUploader* uploader = bitmap_uploaders_[view_id];
+  if (!uploader) {
+    SurfacesServicePtr surfaces_service;
+    InterfacePtr<ServiceProvider> surfaces_service_provider;
+    shell_->ConnectToApplication("mojo:mojo_surfaces_service",
+                                 Get(&surfaces_service_provider));
+    ConnectToService(surfaces_service_provider.get(), &surfaces_service);
+    GpuPtr gpu_service;
+    InterfacePtr<ServiceProvider> gpu_service_provider;
+    shell_->ConnectToApplication("mojo:mojo_native_viewport_service",
+                                 Get(&gpu_service_provider));
+    ConnectToService(gpu_service_provider.get(), &gpu_service);
+    uploader = new BitmapUploader(
+        this, view_id, surfaces_service.Pass(), gpu_service.Pass());
+
+    bitmap_uploaders_[view_id] = uploader;
+  }
+  View* view = GetViewById(view_id);
+  uploader->SetSize(view->bounds().size());
+  return uploader;
+}
+
 }  // namespace mojo