Index: chrome/test/mini_installer_test/chrome_mini_installer.cc |
diff --git a/chrome/test/mini_installer_test/chrome_mini_installer.cc b/chrome/test/mini_installer_test/chrome_mini_installer.cc |
index 27a4227508fadf99d78784e71889e0a2324a8545..a145dd8b3a593f15dc8c150704d4f685e8f1797b 100644 |
--- a/chrome/test/mini_installer_test/chrome_mini_installer.cc |
+++ b/chrome/test/mini_installer_test/chrome_mini_installer.cc |
@@ -183,7 +183,6 @@ void ChromeMiniInstaller::InstallMiniInstaller(bool over_install, |
LOG(INFO) << "Install level is: " |
<< (system_install_ ? "system" : "user"); |
RunInstaller(CommandLine(path)); |
- |
std::string version; |
ASSERT_TRUE(GetChromeVersionFromRegistry(&version)) |
<< "Install failed: unable to get version."; |
@@ -202,7 +201,7 @@ CommandLine ChromeMiniInstaller::GetBaseMultiInstallCommand() { |
void ChromeMiniInstaller::InstallChromeUsingMultiInstall() { |
CommandLine cmd = GetBaseMultiInstallCommand(); |
cmd.AppendSwitch(installer::switches::kChrome); |
- RunInstaller(cmd); |
+ RunInstallerCommand(cmd); |
// Verify installation. |
InstallationValidator::InstallationType type = |
@@ -216,7 +215,11 @@ void ChromeMiniInstaller::InstallChromeUsingMultiInstall() { |
void ChromeMiniInstaller::InstallChromeFrameUsingMultiInstall() { |
CommandLine cmd = GetBaseMultiInstallCommand(); |
- RunInstaller(cmd); |
+ cmd.AppendSwitch(installer::switches::kDoNotCreateShortcuts); |
+ cmd.AppendSwitch(installer::switches::kDoNotLaunchChrome); |
+ cmd.AppendSwitch(installer::switches::kDoNotRegisterForUpdateLaunch); |
+ cmd.AppendSwitch(installer::switches::kChromeFrame); |
+ RunInstallerCommand(cmd); |
// Verify installation. |
InstallationValidator::InstallationType type = |
@@ -241,7 +244,7 @@ void ChromeMiniInstaller::InstallChromeAndChromeFrame(bool ready_mode) { |
cmd.AppendSwitch(installer::switches::kChromeFrame); |
if (ready_mode) |
cmd.AppendSwitch(installer::switches::kChromeFrameReadyMode); |
- RunInstaller(cmd); |
+ RunInstallerCommand(cmd); |
// Verify installation. |
InstallationValidator::InstallationType type = |
installer::ExpectValidInstallation(system_install_); |
@@ -353,7 +356,7 @@ void ChromeMiniInstaller::Repair( |
std::string build_number; |
ASSERT_TRUE(GetChromeVersionFromRegistry(&build_number)); |
FilePath install_path; |
- ASSERT_TRUE(GetChromeInstallDirectoryLocation(&install_path)); |
+ ASSERT_TRUE(GetInstallDirectory(&install_path)); |
install_path = install_path.AppendASCII(build_number); |
ASSERT_TRUE(file_util::Delete(install_path, true)); |
} else if (repair_type == ChromeMiniInstaller::REGISTRY) { |
@@ -460,7 +463,7 @@ void ChromeMiniInstaller::UnInstallChromeFrameWithIERunning() { |
void ChromeMiniInstaller::CleanChromeInstall() { |
DeletePvRegistryKey(); |
FilePath install_path; |
- ASSERT_TRUE(GetChromeInstallDirectoryLocation(&install_path)); |
+ ASSERT_TRUE(GetInstallDirectory(&install_path)); |
ASSERT_TRUE(file_util::Delete(install_path, true)); |
} |
@@ -622,9 +625,9 @@ void ChromeMiniInstaller::FindChromeShortcut() { |
} |
} |
-bool ChromeMiniInstaller::GetChromeInstallDirectoryLocation(FilePath* path) { |
+bool ChromeMiniInstaller::GetInstallDirectory(FilePath* path) { |
BrowserDistribution* dist = GetCurrentBrowserDistribution(); |
- *path = installer::GetChromeInstallPath(system_install_, dist); |
+ *path = installer::GetChromeInstallPath(system_install_, dist); |
FilePath parent; |
if (system_install_) { |
PathService::Get(base::DIR_PROGRAM_FILES, &parent); |
@@ -678,6 +681,12 @@ void ChromeMiniInstaller::RunInstaller(const CommandLine& command) { |
installer.AppendSwitch(installer::switches::kDoNotRegisterForUpdateLaunch); |
installer.AppendSwitch(installer::switches::kChromeFrame); |
} |
+ RunInstallerCommand(installer); |
+} |
+ |
+void ChromeMiniInstaller::RunInstallerCommand(const CommandLine& command) { |
+ ASSERT_TRUE(file_util::PathExists(command.GetProgram())); |
+ CommandLine installer(command); |
if (system_install_) { |
installer.AppendSwitch(installer::switches::kSystemLevel); |
} |
@@ -696,7 +705,7 @@ void ChromeMiniInstaller::LaunchChrome(bool kill) { |
MiniInstallerTestUtil::CloseProcesses(installer::kChromeExe); |
FilePath install_path; |
- ASSERT_TRUE(GetChromeInstallDirectoryLocation(&install_path)); |
+ ASSERT_TRUE(GetInstallDirectory(&install_path)); |
install_path = install_path.Append(installer::kChromeExe); |
CommandLine browser(install_path); |