[Installer] Cleaning up dead code for App Launcher / App Host installs.

chrome/installer/util/channel_info.cc

Index: chrome/installer/util/channel_info.cc
diff --git a/chrome/installer/util/channel_info.cc b/chrome/installer/util/channel_info.cc
index cdf264bad0e11810732746b1948b0259146eea89..2db476f2a5427c1e3e99526e869f968fbbefc05b 100644
--- a/chrome/installer/util/channel_info.cc
+++ b/chrome/installer/util/channel_info.cc
@@ -15,9 +15,6 @@ namespace {
 
 const wchar_t kModChrome[] = L"-chrome";
 const wchar_t kModChromeFrame[] = L"-chromeframe";
-// TODO(huangs): Remove by M27.
-const wchar_t kModAppHostDeprecated[] = L"-apphost";

[grt (UTC plus 2), 2014/12/18 19:27:37]

we're still getting update checks containing -apphost and -applauncher. please
add to the cleanup code that runs on each install/update a call to remove both
values.

[huangs, 2015/01/05 06:01:12]

Updated, with refactoring to ChannelInfo:
- Replaced all std::wstring usage with base::string16.
- Adding static ChannelInfo::AuditValue() (which contains the call to remove
-apphost and -applauncher), which gets called by Initialize() and set_value().
- Moved set_value() etc. from .h to .cc.
- Updated unit tests.

-const wchar_t kModAppLauncher[] = L"-applauncher";
 const wchar_t kModMultiInstall[] = L"-multi";
 const wchar_t kModReadyMode[] = L"-readymode";
 const wchar_t kModStage[] = L"-stage:";
@@ -36,8 +33,6 @@ const wchar_t* const kModifiers[] = {
   kModMultiInstall,
   kModChrome,
   kModChromeFrame,
-  kModAppHostDeprecated,  // TODO(huangs): Remove by M27.
-  kModAppLauncher,
   kModReadyMode,
   kSfxMultiFail,
   kSfxMigrating,
@@ -49,8 +44,6 @@ enum ModifierIndex {
   MOD_MULTI_INSTALL,
   MOD_CHROME,
   MOD_CHROME_FRAME,
-  MOD_APP_HOST_DEPRECATED,  // TODO(huangs): Remove by M27.
-  MOD_APP_LAUNCHER,
   MOD_READY_MODE,
   SFX_MULTI_FAIL,
   SFX_MIGRATING,
@@ -208,17 +201,6 @@ bool ChannelInfo::SetChromeFrame(bool value) {
   return SetModifier(MOD_CHROME_FRAME, value, &value_);
 }
 
-bool ChannelInfo::IsAppLauncher() const {
-  return HasModifier(MOD_APP_LAUNCHER, value_);
-}
-
-bool ChannelInfo::SetAppLauncher(bool value) {
-  // Unconditionally remove -apphost since it has been deprecated.
-  bool changed_app_host = SetModifier(MOD_APP_HOST_DEPRECATED, false, &value_);
-  bool changed_app_launcher = SetModifier(MOD_APP_LAUNCHER, value, &value_);
-  return changed_app_host || changed_app_launcher;
-}
-
 bool ChannelInfo::IsMultiInstall() const {
   return HasModifier(MOD_MULTI_INSTALL, value_);
 }