Implementing unified Chrome / App Launcher flow, and migrating old stand-alone App Launcher.

chrome/installer/setup/install.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/installer/setup/install.h"
 
 #include <windows.h>
 #include <shlobj.h>
 #include <time.h>
 
@@ skipping 229 matching lines @@
              << ", old version: " << (*current_version)->GetString();
 
   return installer::INSTALL_FAILED;
 }
 
 // Deletes the old "Uninstall Google Chrome" shortcut in the Start menu and, if
 // this is a system-level install, also deletes the old Default user Quick
 // Launch shortcut. Both of these were created prior to Chrome 24; in Chrome 24,
 // the uninstall shortcut was removed and the Default user Quick Launch shortcut
 // was replaced by per-user shortcuts created via Active Setup.
-void CleanupLegacyShortcuts(const InstallerState& installer_state,
-                            BrowserDistribution* dist,
-                            const base::FilePath& chrome_exe) {
+void CleanupLegacyChromeShortcuts(const InstallerState& installer_state,
+                                  BrowserDistribution* dist,
+                                  const base::FilePath& chrome_exe) {
   ShellUtil::ShellChange shortcut_level = installer_state.system_install() ?
       ShellUtil::SYSTEM_LEVEL : ShellUtil::CURRENT_USER;
   base::FilePath uninstall_shortcut_path;
   ShellUtil::GetShortcutPath(ShellUtil::SHORTCUT_LOCATION_START_MENU, dist,
                              shortcut_level, &uninstall_shortcut_path);
   uninstall_shortcut_path = uninstall_shortcut_path.Append(
       dist->GetUninstallLinkName() + installer::kLnkExt);
   file_util::Delete(uninstall_shortcut_path, false);
 
   if (installer_state.system_install()) {
@@ skipping 13 matching lines @@
                                      BrowserDistribution::CHROME_APP_HOST);
   bool app_launcher_exists = original_app_host_state &&
       original_app_host_state->uninstall_command()
           .HasSwitch(installer::switches::kChromeAppLauncher);
   if (!app_launcher_exists)
     return installer::INSTALL_SHORTCUT_CREATE_ALL;
 
   return installer::INSTALL_SHORTCUT_REPLACE_EXISTING;
 }
 
+void MigrateLagacyAppLauncherShortcuts(const InstallerState& installer_state) {
+  const base::FilePath app_host_exe(
+      installer_state.target_path().Append(installer::kChromeAppHostExe));
+  const base::FilePath chrome_exe(
+      installer_state.target_path().Append(installer::kChromeExe));
+}

[gab, 2013/04/29 20:25:41]

This doesn't do anything :)

[huangs, 2013/04/30 14:01:49]

This is a stub to call code in
https://chromiumcodereview.appspot.com/14287008/

+
 }  // end namespace
 
 namespace installer {
 
 void EscapeXmlAttributeValueInSingleQuotes(string16* att_value) {
   ReplaceChars(*att_value, L"&", L"&", att_value);
   ReplaceChars(*att_value, L"'", L"'", att_value);
   ReplaceChars(*att_value, L"<", L"&lt;", att_value);
 }
 
@@ skipping 229 matching lines @@
     // installed and for the binaries in case of multi-install.
     installer_state.UpdateChannels();
 
     installer_state.UpdateStage(installer::COPYING_PREFERENCES_FILE);
 
     if (result == FIRST_INSTALL_SUCCESS && !prefs_path.empty())
       CopyPreferenceFileForFirstRun(installer_state, prefs_path);
 
     installer_state.UpdateStage(installer::CREATING_SHORTCUTS);
 
+    InstallShortcutLevel install_level = installer_state.system_install() ?
+        ALL_USERS : CURRENT_USER;
+      const base::FilePath chrome_exe(

[gab, 2013/04/29 20:25:41]

indent

[huangs, 2013/04/30 14:01:49]

Done.

+          installer_state.target_path().Append(kChromeExe));
+
     const Product* app_launcher_product =
         installer_state.FindProduct(BrowserDistribution::CHROME_APP_HOST);
     // Creates shortcuts for App Launcher.
     if (app_launcher_product) {
-      // TODO(huangs): Remove this check once we have system-level App Host.
-      DCHECK(!installer_state.system_install());
-      const base::FilePath app_host_exe(
-          installer_state.target_path().Append(kChromeAppHostExe));
-      InstallShortcutOperation app_launcher_shortcut_operation =
-          GetAppLauncherShortcutOperation(original_state, installer_state);
-
-      // Always install per-user shortcuts for App Launcher.
-      CreateOrUpdateShortcuts(app_host_exe, *app_launcher_product, prefs,
-                              CURRENT_USER, app_launcher_shortcut_operation);
+      // If we have a legacy user-level App Launcher that uses app_host.exe,
+      // then migrate shortcuts.
+      if (!installer_state.system_install()) {
+        const ProductState* orig_product = original_state.GetProductState(
+            false, BrowserDistribution::CHROME_APP_HOST);
+        if (orig_product && chrome_launcher_support::HasLegacyAppHostExe(
+                orig_product->version())) {
+          MigrateLagacyAppLauncherShortcuts(installer_state);
+        }
+      }
+      if (installer_state.create_app_launcher_shortcuts()) {
+        InstallShortcutOperation app_launcher_shortcut_operation =
+            GetAppLauncherShortcutOperation(original_state, installer_state);
+        CreateOrUpdateShortcuts(chrome_exe, *app_launcher_product, prefs,
+                                install_level, app_launcher_shortcut_operation);

[gab, 2013/04/29 20:25:41]

Wait, are we creating All-users app launcher shortcuts?!

[huangs, 2013/04/30 14:01:49]

Chrome and App Launcher are unified, so whatever Chrome shortcut does, App
Launcher shortcut should follow; this includes system-level presence.  For M28
this will need a switch (hence .create_app_launcher_shortcuts()).  For M29 this 
is planned to appear always.

+      }
     }
 
     const Product* chrome_product =
         installer_state.FindProduct(BrowserDistribution::CHROME_BROWSER);
     // Creates shortcuts for Chrome.
     if (chrome_product) {
       BrowserDistribution* chrome_dist = chrome_product->distribution();
-      const base::FilePath chrome_exe(
-          installer_state.target_path().Append(kChromeExe));
-      CleanupLegacyShortcuts(installer_state, chrome_dist, chrome_exe);
+      CleanupLegacyChromeShortcuts(installer_state, chrome_dist, chrome_exe);
 
       // Install per-user shortcuts on user-level installs and all-users

[gab, 2013/04/29 20:25:41]

This comment belonged with the code that moved.

[huangs, 2013/04/30 14:01:49]

I thought this refers to CreateOrUpdateShortcuts() below??|install_level| is a
generic thing, which is needed by my code, therefore moved.

       // shortcuts on system-level installs. Note that Active Setup will take
       // care of installing missing per-user shortcuts on system-level install
       // (i.e., quick launch, taskbar pin, and possibly deleted all-users
       // shortcuts).
-      InstallShortcutLevel install_level = installer_state.system_install() ?
-          ALL_USERS : CURRENT_USER;
-
       InstallShortcutOperation install_operation =
           INSTALL_SHORTCUT_REPLACE_EXISTING;
       if (result == installer::FIRST_INSTALL_SUCCESS ||
           result == installer::INSTALL_REPAIRED ||
           !original_state.GetProductState(installer_state.system_install(),
                                           chrome_dist->GetType())) {
         // Always create the shortcuts on a new install, a repair install, and
         // when the Chrome product is being added to the current install.
         install_operation = INSTALL_SHORTCUT_CREATE_ALL;
       }
@@ skipping 111 matching lines @@
   if (app_host_path.empty())
     return false;
 
   CommandLine cmd(app_host_path);
   cmd.AppendSwitchASCII(::switches::kInstallFromWebstore, app_code);
   VLOG(1) << "App install command: " << cmd.GetCommandLineString();
   return base::LaunchProcess(cmd, base::LaunchOptions(), NULL);
 }
 
 }  // namespace installer