Support an --install-from-webstore command line switch

chrome/browser/extensions/startup_helper.cc

 // Copyright (c) 2012 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/extensions/startup_helper.h"
 
+#include "base/bind.h"
 #include "base/command_line.h"
+#include "base/message_loop.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/extension_service.h"
+#include "chrome/browser/extensions/webstore_inline_installer.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/chrome_switches.h"
+#include "content/public/browser/web_contents.h"
+#include "ipc/ipc_message.h"
 
 namespace {
 
 void PrintPackExtensionMessage(const std::string& message) {
   base::StringPrintf("%s\n", message.c_str());
 }
 
 }  // namespace
 
 namespace extensions {
@@ skipping 45 matching lines @@
   ExtensionService* extension_service = profile->GetExtensionService();
   if (!extension_service)
     return false;
 
   std::string extension_id = cmd_line.GetSwitchValueASCII(
       switches::kUninstallExtension);
   return ExtensionService::UninstallExtensionHelper(extension_service,
                                                     extension_id);
 }
 
+namespace {
+
+class AppInstallHelper {
+ public:
+  AppInstallHelper();
+  virtual ~AppInstallHelper();
+
+  WebstoreInlineInstaller::Callback Callback();
+  void OnAppInstallComplete(bool success, const std::string& error);
+};
+
+AppInstallHelper::AppInstallHelper() {}
+
+AppInstallHelper::~AppInstallHelper() {}
+
+WebstoreInlineInstaller::Callback AppInstallHelper::Callback() {
+  return base::Bind(&AppInstallHelper::OnAppInstallComplete,

[Mihai Parparita -not on Chrome, 2012/09/07 22:01:36]

Why not do this inline in te WebstoreInlineInstaller call?

[asargent_no_longer_on_chrome, 2012/09/14 23:24:35]

Done.

+                    base::Unretained(this));
+}
+void AppInstallHelper::OnAppInstallComplete(bool success,
+                                            const std::string& error) {

[Mihai Parparita -not on Chrome, 2012/09/07 22:01:36]

It would be nice to somehow propagate this error back to the return value of
InstallAppFromId.

[asargent_no_longer_on_chrome, 2012/09/14 23:24:35]

Done.

+  MessageLoop::current()->Quit();
+}
+
+}  // namespace
+
+bool StartupHelper::InstallAppFromID(const std::string& id, Profile* profile) {
+  scoped_ptr<content::WebContents> web_contents(

[Mihai Parparita -not on Chrome, 2012/09/07 22:01:36]

Add a TODO about removing the need for this WebContents (I thought we had
discussed how it wasn't strictly necessary)?

[asargent_no_longer_on_chrome, 2012/09/14 23:24:35]

Done.

+      content::WebContents::Create(profile, NULL, MSG_ROUTING_NONE, NULL));
+
+  AppInstallHelper helper;
+  scoped_refptr<WebstoreInlineInstaller> installer(
+      new WebstoreInlineInstaller(
+          web_contents.get(), id, false, GURL(), helper.Callback()));
+  installer->BeginInstall();
+
+  MessageLoop::current()->Run();
+  return false;
+}
+
 StartupHelper::~StartupHelper() {
   if (pack_job_.get())
     pack_job_->ClearClient();
 }
 
 }  // namespace extensions