Removing non-Launcher Apps V2 flow, interpreting --app-host as --app-launcher.

The non-Launcher Apps V2 flow:
- Was installed by the "--app-host" switch, and has ap value "-apphost".
- Enables V2 apps to appear in the NTP.
- Cannot be directly uninstalled.

Keeping it around increased complexity, and led to "missed-a-spot" bugs, such as Issue 178025 where app_host.exe passes "--app-host" to setup.exe when app_host.exe updates itself.

Now we remove this flow from the installer, and not keep track of the distinction between "App Host" and "App Launcher".

However, for backwards compatibility, setup.exe will still interpret "--app-host" as "--app-launcher", but will not genreate commands with "--app-host".

BUG=178025,178834


Review URL: https://chromiumcodereview.appspot.com/12388022

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@185432 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 2 deletions

diff --git a/apps/app_host/update.cc b/apps/app_host/update.cc
index e8b14be8..ee97dfb 100644
--- a/apps/app_host/update.cc
+++ b/apps/app_host/update.cc
@@ -25,7 +25,7 @@ const wchar_t kGoogleRegClientsKey[] = L"Software\\Google\\Update\\Clients\\";
 
 // Copied from util_constants.cc.
 const char kMultiInstall[] = "multi-install";
-const char kChromeAppHost[] = "app-host";
+const char kChromeAppLauncher[] = "app-launcher";
 const char kVerboseLogging[] = "verbose-logging";
 
 // Copied from google_update_constants.cc.
@@ -75,7 +75,7 @@ bool LaunchAppHostUpdate() {
   }
   CommandLine cmd_line(setup_exe);
   cmd_line.AppendSwitch(kMultiInstall);
-  cmd_line.AppendSwitch(kChromeAppHost);
+  cmd_line.AppendSwitch(kChromeAppLauncher);
   cmd_line.AppendSwitch(kVerboseLogging);
   LOG(INFO) << "Launching: " << cmd_line.GetCommandLineString();
   return base::LaunchProcess(cmd_line, base::LaunchOptions(), NULL);