Add a private API method to install a webstore app from Files.app

chrome/browser/chromeos/extensions/file_manager/private_api_misc.cc

 // Copyright 2013 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/chromeos/extensions/file_manager/private_api_misc.h"
 
 #include "base/files/file_path.h"
 #include "base/prefs/pref_service.h"
 #include "base/values.h"
 #include "chrome/browser/chromeos/drive/drive_integration_service.h"
 #include "chrome/browser/chromeos/drive/logging.h"
+#include "chrome/browser/chromeos/extensions/file_manager/file_manager_installer.h"
 #include "chrome/browser/chromeos/extensions/file_manager/private_api_util.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/render_view_host.h"
 #include "content/public/common/page_zoom.h"
 #include "url/gurl.h"
 
 namespace file_manager {
@@ skipping 169 matching lines @@
   } else if (operation == "reset") {
     zoom_type = content::PAGE_ZOOM_RESET;
   } else {
     NOTREACHED();
     return false;
   }
   view_host->Zoom(zoom_type);
   return true;
 }
 
+InstallWebstoreItemFunction::InstallWebstoreItemFunction() {
+}
+
+InstallWebstoreItemFunction::~InstallWebstoreItemFunction() {
+}
+
+bool InstallWebstoreItemFunction::RunImpl() {
+  if (args_->GetSize() < 1) {

[hashimoto, 2013/08/30 09:09:37]

nit: No need to have '{'?

[yoshiki, 2013/08/30 09:51:46]

Removed.

+    return false;
+  }
+
+  if (!args_->GetString(0, &webstore_item_id_) || webstore_item_id_.empty())
+    return false;
+
+  extensions::WebstoreStandaloneInstaller::Callback callback =
+      base::Bind(&InstallWebstoreItemFunction::OnInstallComplete,
+                 base::Unretained(this));
+
+  scoped_refptr<FileManagerInstaller> installer(
+      new FileManagerInstaller(
+          GetAssociatedWebContents(),  // web_contents(),
+          webstore_item_id_,
+          profile(),
+          callback));
+  // installer will be AddRef()'d in BeginInstall().
+  installer->BeginInstall();
+
+  AddRef();  // Balanced in Release() in OnInstallComplete.

[hashimoto, 2013/08/30 09:09:37]

How about binding |this| to the callback without base::Unretained instead of
calling AddRef manually?

[yoshiki, 2013/08/30 09:51:46]

Nice. Done.

+  return true;
+}
+
+void InstallWebstoreItemFunction::OnInstallComplete(bool success,
+                                                   const std::string& error) {

[hashimoto, 2013/08/30 09:09:37]

nit: indent.

[yoshiki, 2013/08/30 09:51:46]

Done.

+  if (success) {
+    drive::util::Log(logging::LOG_INFO,
+                     "App install succeeded. (item id: %s)",
+                     webstore_item_id_.c_str());
+  } else {
+    drive::util::Log(logging::LOG_ERROR,
+                     "App install failed. (item id: %s, reason: %s)",
+                     webstore_item_id_.c_str(),
+                     error.c_str());
+    error_ = error;
+  }
+
+  SendResponse(success);
+
+  Release();  // Matches the AddRef in RunImpl.
+}
+
 }  // namespace file_manager