| Index: chrome/installer/setup/installer_state_unittest.cc
|
| diff --git a/chrome/installer/setup/installer_state_unittest.cc b/chrome/installer/setup/installer_state_unittest.cc
|
| index c34c3a09f793c5da60d6fc05f64a03dfd0d98c65..9f82c5c2e737a0473ec319f0332a2dfea86b749d 100644
|
| --- a/chrome/installer/setup/installer_state_unittest.cc
|
| +++ b/chrome/installer/setup/installer_state_unittest.cc
|
| @@ -123,12 +123,16 @@ TEST_F(InstallerStateTest, InstallerResult) {
|
| std::wstring value;
|
| DWORD dw_value;
|
|
|
| - // check results for a fresh install of single Chrome
|
| - {
|
| + // Check results for a fresh install of single Chrome and the same for an
|
| + // attempt at multi-install, which is now ignored.
|
| + static constexpr const wchar_t* kCommandLines[] = {
|
| + L"setup.exe --system-level",
|
| + L"setup.exe --system-level --multi-install --chrome",
|
| + };
|
| + for (const wchar_t* command_line : kCommandLines) {
|
| RegistryOverrideManager override_manager;
|
| override_manager.OverrideRegistry(root);
|
| - base::CommandLine cmd_line =
|
| - base::CommandLine::FromString(L"setup.exe --system-level");
|
| + base::CommandLine cmd_line = base::CommandLine::FromString(command_line);
|
| const MasterPreferences prefs(cmd_line);
|
| InstallationState machine_state;
|
| machine_state.Initialize();
|
| @@ -154,38 +158,6 @@ TEST_F(InstallerStateTest, InstallerResult) {
|
| key.ReadValue(installer::kInstallerSuccessLaunchCmdLine, &value));
|
| EXPECT_EQ(launch_cmd, value);
|
| }
|
| -
|
| - // check results for a fresh install of multi Chrome
|
| - {
|
| - RegistryOverrideManager override_manager;
|
| - override_manager.OverrideRegistry(root);
|
| - base::CommandLine cmd_line = base::CommandLine::FromString(
|
| - L"setup.exe --system-level --multi-install --chrome");
|
| - const MasterPreferences prefs(cmd_line);
|
| - InstallationState machine_state;
|
| - machine_state.Initialize();
|
| - InstallerState state;
|
| - state.Initialize(cmd_line, prefs, machine_state);
|
| - state.WriteInstallerResult(installer::FIRST_INSTALL_SUCCESS, 0,
|
| - &launch_cmd);
|
| - BrowserDistribution* distribution =
|
| - BrowserDistribution::GetSpecificDistribution(
|
| - BrowserDistribution::CHROME_BROWSER);
|
| - BrowserDistribution* binaries =
|
| - BrowserDistribution::GetSpecificDistribution(
|
| - BrowserDistribution::CHROME_BINARIES);
|
| - EXPECT_EQ(ERROR_SUCCESS,
|
| - key.Open(root, distribution->GetStateKey().c_str(), KEY_READ));
|
| - EXPECT_EQ(ERROR_SUCCESS,
|
| - key.ReadValue(installer::kInstallerSuccessLaunchCmdLine, &value));
|
| - EXPECT_EQ(launch_cmd, value);
|
| - EXPECT_EQ(ERROR_SUCCESS,
|
| - key.Open(root, binaries->GetStateKey().c_str(), KEY_READ));
|
| - EXPECT_EQ(ERROR_SUCCESS,
|
| - key.ReadValue(installer::kInstallerSuccessLaunchCmdLine, &value));
|
| - EXPECT_EQ(launch_cmd, value);
|
| - key.Close();
|
| - }
|
| }
|
|
|
| // Test GetCurrentVersion when migrating single Chrome to multi
|
| @@ -259,27 +231,27 @@ TEST_F(InstallerStateTest, InitializeTwice) {
|
|
|
| InstallerState installer_state;
|
|
|
| - // Initialize the instance to install multi Chrome.
|
| + // Initialize the instance to install user-level Chrome.
|
| {
|
| - base::CommandLine cmd_line(
|
| - base::CommandLine::FromString(L"setup.exe --multi-install --chrome"));
|
| + base::CommandLine cmd_line(base::CommandLine::FromString(L"setup.exe"));
|
| MasterPreferences prefs(cmd_line);
|
| installer_state.Initialize(cmd_line, prefs, machine_state);
|
| }
|
| // Confirm the expected state.
|
| EXPECT_EQ(InstallerState::USER_LEVEL, installer_state.level());
|
| - EXPECT_EQ(InstallerState::MULTI_PACKAGE, installer_state.package_type());
|
| - EXPECT_EQ(InstallerState::MULTI_INSTALL, installer_state.operation());
|
| + EXPECT_EQ(InstallerState::SINGLE_PACKAGE, installer_state.package_type());
|
| + EXPECT_EQ(InstallerState::SINGLE_INSTALL_OR_UPDATE,
|
| + installer_state.operation());
|
| EXPECT_TRUE(wcsstr(installer_state.target_path().value().c_str(),
|
| BrowserDistribution::GetSpecificDistribution(
|
| - BrowserDistribution::CHROME_BINARIES)->
|
| - GetInstallSubDir().c_str()));
|
| + BrowserDistribution::CHROME_BROWSER)
|
| + ->GetInstallSubDir()
|
| + .c_str()));
|
| EXPECT_FALSE(installer_state.verbose_logging());
|
| EXPECT_EQ(installer_state.state_key(),
|
| BrowserDistribution::GetSpecificDistribution(
|
| BrowserDistribution::CHROME_BROWSER)->GetStateKey());
|
| EXPECT_EQ(installer_state.state_type(), BrowserDistribution::CHROME_BROWSER);
|
| - EXPECT_TRUE(installer_state.multi_package_binaries_distribution());
|
| EXPECT_TRUE(installer_state.FindProduct(BrowserDistribution::CHROME_BROWSER));
|
|
|
| // Now initialize it to install system-level single Chrome.
|
|
|