Make application shortcuts point to app_host.exe, install App Host during app installation.

chrome/browser/extensions/crx_installer.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_CRX_INSTALLER_H_
 #define CHROME_BROWSER_EXTENSIONS_CRX_INSTALLER_H_
 
 #include <string>
 
 #include "base/compiler_specific.h"
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/version.h"
 #include "chrome/browser/extensions/crx_installer_error.h"
 #include "chrome/browser/extensions/extension_install_prompt.h"
 #include "chrome/browser/extensions/sandboxed_unpacker.h"
 #include "chrome/browser/extensions/webstore_installer.h"
 #include "chrome/common/extensions/extension.h"
 #include "chrome/common/web_apps.h"
 #include "sync/api/string_ordinal.h"
 
+#if defined(OS_WIN)
+#include "chrome/browser/extensions/app_host_installer_win.h"
+#endif
+
 class ExtensionService;
 class SkBitmap;
 
 namespace extensions {
 class ExtensionUpdaterTest;
 class RequirementsChecker;
 
 // This class installs a crx file into a profile.
 //
 // Installing a CRX is a multi-step process, including unpacking the crx,
@@ skipping 174 matching lines @@
   // Called on the UI thread to start the requirements check on the extension.
   void CheckRequirements();
 
   // Runs on the UI thread. Callback from RequirementsChecker.
   void OnRequirementsChecked(std::vector<std::string> requirement_errors);
 
   // Runs on the UI thread. Confirms with the user (via ExtensionInstallPrompt)
   // that it is OK to install this extension.
   void ConfirmInstall();
 
-  // Runs on File thread. Install the unpacked extension into the profile and
-  // notify the frontend.
+  // Runs on File thread. Ensures that App Host is installed, and then
+  // proceeds to post CompleteExtensionInstall().
   void CompleteInstall();
 
+  // Runs on File thread. Installs the unpacked extension into the profile and
+  // notifies the frontend.
+  void CompleteExtensionInstall();

[benwells, 2012/10/03 05:08:43]

Please come up with a better name for this function. There is CompleteInstall
and CompleteExtensionInstall, how are they different?

From a quick glance, maybe CompleteInstall should be CheckEnvironmentAndInstall,
and CompleteExtensionInstall CompleteInstall. Or maybe you can come up with
something better.

[erikwright (departed), 2012/10/03 15:58:35]

Naming is hell.

How about

CompleteInstall->BeginInstall
CompleteExtensionInstall->CompleteInstall

[huangs, 2012/10/03 20:09:47]

Done.

[huangs, 2012/10/03 20:09:47]

Done.

+
+#if defined(OS_WIN)
+  void OnAppHostInstallFailure();
+#endif
+
   // Result reporting.
   void ReportFailureFromFileThread(const CrxInstallerError& error);
   void ReportFailureFromUIThread(const CrxInstallerError& error);
   void ReportSuccessFromFileThread();
   void ReportSuccessFromUIThread();
   void NotifyCrxInstallComplete(const Extension* extension);
 
   // The file we're installing.
   FilePath source_file_;
 
@@ skipping 121 matching lines @@
 
   // Whether we should produce an error if the manifest declares requirements
   // that are not met. If false and there is an unmet requirement, the install
   // will continue but the extension will be distabled.
   bool error_on_unsupported_requirements_;
 
   scoped_ptr<RequirementsChecker> requirements_checker_;
 
   bool has_requirement_errors_;
 
+#if defined(OS_WIN)
+  AppHostInstaller app_host_installer_;
+#endif
+
   DISALLOW_COPY_AND_ASSIGN(CrxInstaller);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_CRX_INSTALLER_H_