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/app_host_installer.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"
 
 class ExtensionService;
 class SkBitmap;
@@ skipping 182 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

[benwells, 2012/10/04 07:53:01]

Nit: File -> FILE.

Nit: This function does more than what the comment describes (it also checks
minimum version), so the comment is a little misleading as is.

[huangs, 2012/10/04 22:56:42]

Done.  Also fixed every instance of "file thread" in file.

+  // proceeds to post CompleteExtensionInstall().

[erikwright (departed), 2012/10/04 01:28:16]

CompleteInstall

[huangs, 2012/10/04 22:56:42]

Done.

+  void BeginInstall();
+
+  // Callback founction for AppHostInstaller.

[erikwright (departed), 2012/10/04 01:28:16]

founction

[huangs, 2012/10/04 22:56:42]

Done.

+  void OnAppHostInstallation(bool success);
+

[benwells, 2012/10/04 07:53:01]

Nit: File -> FILE

[huangs, 2012/10/04 22:56:42]

Done.

+  // Runs on File thread. Installs the unpacked extension into the profile and
+  // notifies the frontend.
   void CompleteInstall();
 
   // 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.
@@ skipping 123 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_;
 
+  // Helper to install App Host.
+  AppHostInstaller app_host_installer_;
+
   DISALLOW_COPY_AND_ASSIGN(CrxInstaller);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_CRX_INSTALLER_H_