Support uninstalling ARC app from Chrome launcher

chrome/browser/ui/app_list/arc/arc_app_utils.cc

 // Copyright 2016 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 "chrome/browser/ui/app_list/arc/arc_app_utils.h"
 
 #include "chrome/browser/ui/app_list/arc/arc_app_list_prefs.h"
 #include "components/arc/arc_bridge_service.h"
+#include "mojo/public/cpp/bindings/string.h"
 
 namespace arc {
 
 void GetAndroidAppTargetRect(gfx::Rect& target_rect) {
   // TODO: Figure out where to put the android window.
   target_rect.SetRect(0, 0, 0, 0);
 }
 
 bool LaunchApp(content::BrowserContext* context, const std::string& app_id) {
   ArcAppListPrefs* prefs = ArcAppListPrefs::Get(context);
   CHECK(prefs);
 
   scoped_ptr<ArcAppListPrefs::AppInfo> app_info = prefs->GetApp(app_id);
   if (!app_info) {
-    VLOG(2) << "Cannot launch unavailable app:" << app_id << ".";
+    VLOG(2) << "Cannot launch unavailable app: " << app_id << ".";
     return false;
   }
 
   if (!app_info->ready) {
-    VLOG(2) << "Cannot launch not-ready app:" << app_id << ".";
+    VLOG(2) << "Cannot launch not-ready app: " << app_id << ".";
     return false;
   }
 
   arc::ArcBridgeService* bridge_service = arc::ArcBridgeService::Get();
   if (!bridge_service) {
     VLOG(2) << "Request to launch app when bridge service is not ready: "
             << app_id << ".";
     return false;
   }
   arc::AppInstance* app_instance = bridge_service->app_instance();
@@ skipping 15 matching lines @@
   app_instance->LaunchApp(app_info->package_name, app_info->activity,
                           std::move(rect));
   prefs->SetLastLaunchTime(app_id, base::Time::Now());
 
   return true;
 }
 
 bool CanHandleResolution(content::BrowserContext* context,
     const std::string& app_id,
     const gfx::Rect& rect,
-    const CanHandleResolutionCallback callback) {
+    const CanHandleResolutionCallback& callback) {
   ArcAppListPrefs* prefs = ArcAppListPrefs::Get(context);
   scoped_ptr<ArcAppListPrefs::AppInfo> app_info = prefs->GetApp(app_id);
   if (!app_info) {
     VLOG(2) << "Cannot test resolution capability of unavailable app:" << app_id
             << ".";
     return false;
   }
 
   arc::ArcBridgeService* bridge_service = arc::ArcBridgeService::Get();
   arc::AppInstance* app_instance =
@@ skipping 16 matching lines @@
   screen_rect->right = rect.right();
   screen_rect->top = rect.y();
   screen_rect->bottom = rect.bottom();
 
   app_instance->CanHandleResolution(app_info->package_name, app_info->activity,
                                     std::move(screen_rect),
                                     callback);
   return true;
 }
 
+bool UninstallApp(const std::string& package_name,
+                  const UninstallAppCallback& callback) {
+  VLOG(2) << "Uninstalling " << package_name;
+  arc::ArcBridgeService* bridge_service = arc::ArcBridgeService::Get();
+  if (!bridge_service) {
+    VLOG(2) << "Request to uninstall app when bridge service is not ready: "
+            << package_name << ".";
+    return false;
+  }
+  arc::AppInstance* app_instance = bridge_service->app_instance();
+  if (!app_instance) {
+    VLOG(2) << "Request to uninstall app when bridge service is not ready: "
+            << package_name << ".";
+    return false;
+  }
+
+  if (bridge_service->app_version() < 2) {
+    LOG(ERROR) << "Request to uninstall app when version "
+               << bridge_service->app_version() << " does not support it";
+    return false;
+  }
+
+  app_instance->UninstallApp(package_name, callback);
+  return true;
+}
+
+void OnUninstallAppResponse(const mojo::String& error_message) {
+  if (!error_message.is_null()) {
+    LOG(ERROR) << "Failed to uninstall package: " << error_message.get();

[khmel, 2016/03/08 00:09:54]

nit: if we don't have notification from Android side, please add TODO -> Show
notification or similar case.

[victorhsieh0, 2016/03/08 00:33:07]

Done with TODO.  We should think about a generic way (can just be notification)
to deliver ARC error to the user.

+  }
+}
+
 }  // namespace arc