Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(8582)

Unified Diff: chrome/installer/util/installation_validator_unittest.cc

Issue 10665002: Implement installation of the Chrome App Host. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix unittests. Created 8 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/installer/util/installation_validator_unittest.cc
diff --git a/chrome/installer/util/installation_validator_unittest.cc b/chrome/installer/util/installation_validator_unittest.cc
index 56e854c0e9f26ea236b009bce4409861b3e7e890..c99711004405dc9f07d56107e3de013e1608d36e 100644
--- a/chrome/installer/util/installation_validator_unittest.cc
+++ b/chrome/installer/util/installation_validator_unittest.cc
@@ -78,11 +78,14 @@ class FakeProductState : public ProductState {
const char* version,
int channel_modifiers,
Vehicle vehicle);
+ void AddQuickEnableApplicationHostCommand(BrowserDistribution::Type dist_type,
+ Level install_level,
+ const char* version,
+ int channel_modifiers);
void AddQuickEnableCfCommand(BrowserDistribution::Type dist_type,
Level install_level,
const char* version,
int channel_modifiers);
- void RemoveQuickEnableCfCommand(BrowserDistribution::Type dist_type);
void set_multi_install(bool is_multi_install) {
multi_install_ = is_multi_install;
}
@@ -194,6 +197,23 @@ void FakeProductState::SetUninstallCommand(BrowserDistribution::Type dist_type,
uninstall_command_.AppendSwitch(installer::switches::kMsi);
}
+// Adds the "quick-enable-application-host" Google Update product command.
+void FakeProductState::AddQuickEnableApplicationHostCommand(
+ BrowserDistribution::Type dist_type,
+ Level install_level,
+ const char* version,
+ int channel_modifiers) {
+ DCHECK_EQ(dist_type, BrowserDistribution::CHROME_BINARIES);
+ DCHECK_NE(channel_modifiers & CM_MULTI, 0);
+
+ CommandLine cmd_line(GetSetupExePath(dist_type, install_level, version,
+ channel_modifiers));
+ cmd_line.AppendSwitch(installer::switches::kMultiInstall);
+ cmd_line.AppendSwitch(installer::switches::kChromeAppHost);
+ commands_.Set(installer::kCmdQuickEnableApplicationHost,
+ AppCommand(cmd_line.GetCommandLineString(), true, true));
+}
+
// Adds the "quick-enable-cf" Google Update product command.
void FakeProductState::AddQuickEnableCfCommand(
BrowserDistribution::Type dist_type,
@@ -213,14 +233,6 @@ void FakeProductState::AddQuickEnableCfCommand(
AppCommand(cmd_line.GetCommandLineString(), true, true));
}
-// Removes the "quick-enable-cf" Google Update product command.
-void FakeProductState::RemoveQuickEnableCfCommand(
- BrowserDistribution::Type dist_type) {
- DCHECK_EQ(dist_type, BrowserDistribution::CHROME_BINARIES);
-
- commands_.Remove(installer::kCmdQuickEnableCf);
-}
-
} // namespace
// Fixture for testing the InstallationValidator. Errors logged by the
@@ -412,6 +424,12 @@ void InstallationValidatorTest::MakeProductState(
state->AddQuickEnableCfCommand(prod_type, install_level,
chrome::kChromeVersion, channel_modifiers);
}
+ if (prod_type == BrowserDistribution::CHROME_BINARIES) {
+ state->AddQuickEnableApplicationHostCommand(prod_type,
+ install_level,
+ chrome::kChromeVersion,
+ channel_modifiers);
+ }
}
// static

Powered by Google App Engine
This is Rietveld 408576698