Revert 148046 - Revert 147650 - Implement installation of the Chrome App Host.

chrome/installer/util/installation_validator.cc

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
 // Implementation of the installation validator.
 
 #include "chrome/installer/util/installation_validator.h"
 
 #include <algorithm>
 #include <set>
 
 #include "base/logging.h"
 #include "base/version.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/installer/util/browser_distribution.h"
 #include "chrome/installer/util/helper.h"
 #include "chrome/installer/util/installation_state.h"
 
 namespace installer {
 
 BrowserDistribution::Type
     InstallationValidator::ChromeRules::distribution_type() const {
   return BrowserDistribution::CHROME_BROWSER;
 }
@@ skipping 81 matching lines @@
 }
 
 bool InstallationValidator::ChromeFrameRules::UsageStatsAllowed(
     const ProductState& product_state) const {
   // Products must not have usagestats consent values when multi-install
   // (only the multi-install binaries may).
   return !product_state.is_multi_install();
 }
 
 BrowserDistribution::Type
+    InstallationValidator::ChromeAppHostRules::distribution_type() const {
+  return BrowserDistribution::CHROME_APP_HOST;
+}
+
+void InstallationValidator::ChromeAppHostRules::AddUninstallSwitchExpectations(
+    const InstallationState& machine_state,
+    bool system_install,
+    const ProductState& product_state,
+    SwitchExpectations* expectations) const {
+  DCHECK(!system_install);
+
+  // --chrome-app-host must be present.
+  expectations->push_back(std::make_pair(std::string(switches::kChromeAppHost),
+                                         true));
+  // --chrome must not be present.
+  expectations->push_back(std::make_pair(std::string(switches::kChrome),
+                                         false));
+
+  // --chrome-frame must not be present.
+  expectations->push_back(std::make_pair(std::string(switches::kChromeFrame),
+                                         false));
+}
+
+void InstallationValidator::ChromeAppHostRules::AddRenameSwitchExpectations(
+    const InstallationState& machine_state,
+    bool system_install,
+    const ProductState& product_state,
+    SwitchExpectations* expectations) const {
+  // TODO(erikwright): I guess there will be none?
+}
+
+bool InstallationValidator::ChromeAppHostRules::UsageStatsAllowed(
+    const ProductState& product_state) const {
+  // App Host doesn't manage usage stats. The Chrome Binaries will.
+  return false;
+}
+
+BrowserDistribution::Type
     InstallationValidator::ChromeBinariesRules::distribution_type() const {
   return BrowserDistribution::CHROME_BINARIES;
 }
 
 void InstallationValidator::ChromeBinariesRules::AddUninstallSwitchExpectations(
     const InstallationState& machine_state,
     bool system_install,
     const ProductState& product_state,
     SwitchExpectations* expectations) const {
   NOTREACHED();
@@ skipping 17 matching lines @@
 const InstallationValidator::InstallationType
     InstallationValidator::kInstallationTypes[] = {
   NO_PRODUCTS,
   CHROME_SINGLE,
   CHROME_MULTI,
   CHROME_FRAME_SINGLE,
   CHROME_FRAME_SINGLE_CHROME_SINGLE,
   CHROME_FRAME_SINGLE_CHROME_MULTI,
   CHROME_FRAME_MULTI,
   CHROME_FRAME_MULTI_CHROME_MULTI,
-  CHROME_FRAME_READY_MODE_CHROME_MULTI
+  CHROME_FRAME_READY_MODE_CHROME_MULTI,
+  CHROME_APP_HOST,
+  CHROME_APP_HOST_CHROME_FRAME_SINGLE,
+  CHROME_APP_HOST_CHROME_FRAME_SINGLE_CHROME_MULTI,
+  CHROME_APP_HOST_CHROME_FRAME_MULTI,
+  CHROME_APP_HOST_CHROME_FRAME_MULTI_CHROME_MULTI,
+  CHROME_APP_HOST_CHROME_MULTI,
+  CHROME_APP_HOST_CHROME_MULTI_CHROME_FRAME_READY_MODE,
 };
 
+// Validates the "install-application" Google Update product command.
+void InstallationValidator::ValidateInstallAppCommand(
+    const ProductContext& ctx,
+    const AppCommand& command,
+    bool* is_valid) {
+  DCHECK(is_valid);
+
+  CommandLine the_command(CommandLine::FromString(command.command_line()));
+
+  FilePath expected_path(
+      installer::GetChromeInstallPath(ctx.system_install, ctx.dist)
+      .Append(installer::kChromeAppHostExe));
+
+  if (!FilePath::CompareEqualIgnoreCase(expected_path.value(),
+                                        the_command.GetProgram().value())) {
+    *is_valid = false;
+    LOG(ERROR) << "install-application command's path is not "
+               << expected_path.value() << ": "
+               << the_command.GetProgram().value();
+  }
+
+
+  SwitchExpectations expected;
+
+  expected.push_back(
+      std::make_pair(std::string(::switches::kAppsInstallFromManifestURL),
+                     true));
+
+  ValidateCommandExpectations(ctx, the_command, expected, "install application",
+                              is_valid);
+
+  if (!command.sends_pings()) {
+    *is_valid = false;
+    LOG(ERROR) << "install-application command is not configured to send "
+               << "pings.";
+  }
+
+  if (!command.is_web_accessible()) {
+    *is_valid = false;
+    LOG(ERROR) << "install-application command is not web accessible.";
+  }
+}
+
 // Validates the "quick-enable-cf" Google Update product command.
 void InstallationValidator::ValidateQuickEnableCfCommand(
     const ProductContext& ctx,
     const AppCommand& command,
     bool* is_valid) {
   DCHECK(is_valid);
 
   CommandLine the_command(CommandLine::FromString(command.command_line()));
 
   ValidateSetupPath(ctx, the_command.GetProgram(), "quick enable cf", is_valid);
@@ skipping 14 matching lines @@
     *is_valid = false;
     LOG(ERROR) << "Quick-enable-cf command is not configured to send pings.";
   }
 
   if (!command.is_web_accessible()) {
     *is_valid = false;
     LOG(ERROR) << "Quick-enable-cf command is not web accessible.";
   }
 }
 
+// Validates the "quick-enable-application-host" Google Update product command.
+void InstallationValidator::ValidateQuickEnableApplicationHostCommand(
+    const ProductContext& ctx,
+    const AppCommand& command,
+    bool* is_valid) {
+  DCHECK(is_valid);
+
+  CommandLine the_command(CommandLine::FromString(command.command_line()));
+
+  ValidateSetupPath(ctx,
+                    the_command.GetProgram(),
+                    "quick enable application host",
+                    is_valid);
+
+  SwitchExpectations expected;
+
+  expected.push_back(
+      std::make_pair(std::string(switches::kChromeAppHost), true));
+  expected.push_back(std::make_pair(std::string(switches::kSystemLevel),
+                                    false));
+  expected.push_back(std::make_pair(std::string(switches::kMultiInstall),
+                                    true));
+
+  ValidateCommandExpectations(ctx,
+                              the_command,
+                              expected,
+                              "quick enable application host",
+                              is_valid);
+
+  if (!command.sends_pings()) {
+    *is_valid = false;
+    LOG(ERROR) << "Quick-enable-application-host command is not configured to "
+               << "send pings.";
+  }
+
+  if (!command.is_web_accessible()) {
+    *is_valid = false;
+    LOG(ERROR) << "Quick-enable-application-host command is not web "
+               << "accessible.";
+  }
+}
+
 // Validates a product's set of Google Update product commands against a
 // collection of expectations.
 void InstallationValidator::ValidateAppCommandExpectations(
     const ProductContext& ctx,
     const CommandExpectations& expectations,
     bool* is_valid) {
   DCHECK(is_valid);
 
   CommandExpectations the_expectations(expectations);
 
@@ skipping 26 matching lines @@
                << scan->first << "\".";
   }
 }
 
 // Validates the multi-install binaries' Google Update commands.
 void InstallationValidator::ValidateBinariesCommands(
     const ProductContext& ctx,
     bool* is_valid) {
   DCHECK(is_valid);
 
-  // The quick-enable-cf command must be present if Chrome is installed either
-  // alone or with CF in ready-mode.
+  // The quick-enable-cf command must be present if Chrome Binaries are
+  // installed and Chrome Frame is not installed (or installed in ready mode).
   const ChannelInfo& channel = ctx.state.channel();
-  const ProductState* chrome_state = ctx.machine_state.GetProductState(
-      ctx.system_install, BrowserDistribution::CHROME_BROWSER);
+  const ProductState* binaries_state = ctx.machine_state.GetProductState(
+      ctx.system_install, BrowserDistribution::CHROME_BINARIES);
   const ProductState* cf_state = ctx.machine_state.GetProductState(
       ctx.system_install, BrowserDistribution::CHROME_FRAME);
 
   CommandExpectations expectations;
 
-  if (chrome_state != NULL && (cf_state == NULL || channel.IsReadyMode()))
-    expectations[kCmdQuickEnableCf] = &ValidateQuickEnableCfCommand;
+  if (binaries_state != NULL) {
+    if (cf_state == NULL || channel.IsReadyMode())
+      expectations[kCmdQuickEnableCf] = &ValidateQuickEnableCfCommand;
+
+    expectations[kCmdQuickEnableApplicationHost] =
+        &ValidateQuickEnableApplicationHostCommand;
+  }
 
   ValidateAppCommandExpectations(ctx, expectations, is_valid);
 }
 
 // Validates the multi-install binaries at level |system_level|.
 void InstallationValidator::ValidateBinaries(
     const InstallationState& machine_state,
     bool system_install,
     const ProductState& binaries_state,
     bool* is_valid) {
@@ skipping 46 matching lines @@
       LOG(ERROR) << "Chrome Binaries are missing \"-readymode\" in channel"
                     " name: \"" << channel.value() << "\"";
     }
   } else if (channel.IsReadyMode()) {
     *is_valid = false;
     LOG(ERROR) << "Chrome Binaries have \"-readymode\" in channel name, yet "
                   "Chrome Frame is not in ready mode: \"" << channel.value()
                << "\"";
   }
 
-  // Chrome or Chrome Frame must be present
-  if (chrome_state == NULL && cf_state == NULL) {
+  // ap must have -apphost iff Chrome Frame is installed multi
+  const ProductState* app_host_state = machine_state.GetProductState(
+      system_install, BrowserDistribution::CHROME_APP_HOST);
+  if (app_host_state != NULL) {
+    if (!app_host_state->is_multi_install()) {
+      *is_valid = false;
+      LOG(ERROR) << "Chrome App Host is installed in non-multi mode.";
+    }
+    if (!channel.IsAppHost()) {
+      *is_valid = false;
+      LOG(ERROR) << "Chrome Binaries are missing \"-apphost\" in channel"
+                    " name: \"" << channel.value() << "\"";
+    }
+  } else if (channel.IsAppHost()) {
+    *is_valid = false;
+    LOG(ERROR) << "Chrome Binaries have \"-apphost\" in channel name, yet "
+                  "Chrome App Host is not installed: \"" << channel.value()
+               << "\"";
+  }
+
+  // Chrome, Chrome Frame, or App Host must be present
+  if (chrome_state == NULL && cf_state == NULL && app_host_state == NULL) {
     *is_valid = false;
     LOG(ERROR) << "Chrome Binaries are present with no other products.";
   }
 
   // Chrome must be multi-install if present.
   if (chrome_state != NULL && !chrome_state->is_multi_install()) {
     *is_valid = false;
     LOG(ERROR)
         << "Chrome Binaries are present yet Chrome is not multi-install.";
   }
 
-  // Chrome Frame must be multi-install if Chrome is not present.
-  if (cf_state != NULL && chrome_state == NULL &&
+  // Chrome Frame must be multi-install if Chrome & App Host are not present.
+  if (cf_state != NULL && app_host_state == NULL && chrome_state == NULL &&
       !cf_state->is_multi_install()) {
     *is_valid = false;
-    LOG(ERROR) << "Chrome Binaries are present without Chrome yet Chrome Frame "
-                  "is not multi-install.";
+    LOG(ERROR) << "Chrome Binaries are present without Chrome nor App Host "
+               << "yet Chrome Frame is not multi-install.";
   }
 
   ChromeBinariesRules binaries_rules;
   ProductContext ctx = {
     machine_state,
     system_install,
     BrowserDistribution::GetSpecificDistribution(
         BrowserDistribution::CHROME_BINARIES),
     binaries_state,
     binaries_rules
@@ skipping 134 matching lines @@
 
 // Validates the multi-install state of the product described in |ctx|.
 void InstallationValidator::ValidateMultiInstallProduct(
     const ProductContext& ctx,
     bool* is_valid) {
   DCHECK(is_valid);
 
   const ProductState* binaries =
       ctx.machine_state.GetProductState(ctx.system_install,
                                         BrowserDistribution::CHROME_BINARIES);
-  DCHECK(binaries);
+  if (!binaries) {
+    if (ctx.dist->GetType() == BrowserDistribution::CHROME_APP_HOST) {
+      if (!ctx.machine_state.GetProductState(
+              true,  // system-level
+              BrowserDistribution::CHROME_BINARIES) &&
+          !ctx.machine_state.GetProductState(
+              true,  // system-level
+              BrowserDistribution::CHROME_BROWSER)) {
+        *is_valid = false;
+        LOG(ERROR) << ctx.dist->GetAppShortCutName()
+                   << " (" << ctx.state.version().GetString() << ") is "
+                   << "installed without Chrome Binaries or a system-level "
+                   << "Chrome.";
+      }
+    } else {
+      *is_valid = false;
+      LOG(ERROR) << ctx.dist->GetAppShortCutName()
+                 << " (" << ctx.state.version().GetString() << ") is installed "
+                 << "without Chrome Binaries.";
+    }
+  } else {
+    // Version must match that of binaries.
+    if (ctx.state.version().CompareTo(binaries->version()) != 0) {
+      *is_valid = false;
+      LOG(ERROR) << "Version of " << ctx.dist->GetAppShortCutName()
+                 << " (" << ctx.state.version().GetString() << ") does not "
+                    "match that of Chrome Binaries ("
+                 << binaries->version().GetString() << ").";
+    }
 
-  // Version must match that of binaries.
-  if (ctx.state.version().CompareTo(binaries->version()) != 0) {
-    *is_valid = false;
-    LOG(ERROR) << "Version of " << ctx.dist->GetAppShortCutName()
-               << " (" << ctx.state.version().GetString() << ") does not "
-                  "match that of Chrome Binaries ("
-               << binaries->version().GetString() << ").";
-  }
-
-  // Channel value must match that of binaries.
-  if (!ctx.state.channel().Equals(binaries->channel())) {
-    *is_valid = false;
-    LOG(ERROR) << "Channel name of " << ctx.dist->GetAppShortCutName()
-               << " (" << ctx.state.channel().value()
-               << ") does not match that of Chrome Binaries ("
-               << binaries->channel().value() << ").";
+    // Channel value must match that of binaries.
+    if (!ctx.state.channel().Equals(binaries->channel())) {
+      *is_valid = false;
+      LOG(ERROR) << "Channel name of " << ctx.dist->GetAppShortCutName()
+                 << " (" << ctx.state.channel().value()
+                 << ") does not match that of Chrome Binaries ("
+                 << binaries->channel().value() << ").";
+    }
   }
 }
 
 // Validates the Google Update commands for the product described in |ctx|.
 void InstallationValidator::ValidateAppCommands(
     const ProductContext& ctx,
     bool* is_valid) {
   DCHECK(is_valid);
 
-  // Products are not expected to have any commands.
-  ValidateAppCommandExpectations(ctx, CommandExpectations(), is_valid);
+  CommandExpectations expectations;
+
+  if (ctx.dist->GetType() == BrowserDistribution::CHROME_APP_HOST) {
+    expectations[kCmdInstallApp] = &ValidateInstallAppCommand;
+  }
+
+  ValidateAppCommandExpectations(ctx, expectations, is_valid);
 }
 
 // Validates usagestats for the product or binaries in |ctx|.
 void InstallationValidator::ValidateUsageStats(const ProductContext& ctx,
                                                bool* is_valid) {
   DWORD usagestats = 0;
   if (ctx.state.GetUsageStats(&usagestats)) {
     if (!ctx.rules.UsageStatsAllowed(ctx.state)) {
       *is_valid = false;
       LOG(ERROR) << ctx.dist->GetAppShortCutName()
@@ skipping 77 matching lines @@
                     chrome_frame_rules, &rock_on);
     int cf_bit = !product_state->is_multi_install() ?
         ProductBits::CHROME_FRAME_SINGLE :
         (product_state->uninstall_command().HasSwitch(
              switches::kChromeFrameReadyMode) ?
                  ProductBits::CHROME_FRAME_READY_MODE :
                  ProductBits::CHROME_FRAME_MULTI);
     *type = static_cast<InstallationType>(*type | cf_bit);
   }
 
+  // Is Chrome App Host installed?
+  product_state =
+      machine_state.GetProductState(system_level,
+                                    BrowserDistribution::CHROME_APP_HOST);
+  if (product_state != NULL) {
+    ChromeAppHostRules chrome_app_host_rules;
+    ValidateProduct(machine_state, system_level, *product_state,
+                    chrome_app_host_rules, &rock_on);
+    *type = static_cast<InstallationType>(*type | ProductBits::CHROME_APP_HOST);
+    if (system_level) {
+      LOG(ERROR) << "Chrome App Host must not be installed at system level.";
+      rock_on = false;
+    }
+    if (!product_state->is_multi_install()) {
+      LOG(ERROR) << "Chrome App Host must always be multi-install.";
+      rock_on = false;
+    }
+  }
+
   DCHECK_NE(std::find(&kInstallationTypes[0],
                       &kInstallationTypes[arraysize(kInstallationTypes)],
                       *type),
             &kInstallationTypes[arraysize(kInstallationTypes)])
       << "Invalid combination of products found on system (" << *type << ")";
 
   return rock_on;
 }
 
 // static
 bool InstallationValidator::ValidateInstallationType(bool system_level,
                                                      InstallationType* type) {
   DCHECK(type);
   InstallationState machine_state;
 
   machine_state.Initialize();
 
   return ValidateInstallationTypeForState(machine_state, system_level, type);
 }
 
 }  // namespace installer