Chromium Code Reviews| Index: chrome/test/mini_installer_test/test.cc |
| diff --git a/chrome/test/mini_installer_test/test.cc b/chrome/test/mini_installer_test/test.cc |
| index b5b2a2a1e6c7dbe847c45845e227e7a6d65fcc33..3a5547e991e69f65d4e894ddee593e400ef8fc13 100644 |
| --- a/chrome/test/mini_installer_test/test.cc |
| +++ b/chrome/test/mini_installer_test/test.cc |
| @@ -27,15 +27,12 @@ class MiniInstallTest : public testing::Test { |
| static void CleanTheSystem() { |
| const CommandLine* cmd = CommandLine::ForCurrentProcess(); |
| if (cmd->HasSwitch(installer::switches::kChromeFrame)) { |
| - ChromeMiniInstaller systeminstall(kSystemInstall, |
| - cmd->HasSwitch(installer::switches::kChromeFrame)); |
| + ChromeMiniInstaller systeminstall(true, true); |
|
kkania
2011/09/28 18:02:43
this can be simplified by just ChromeMiniInstaller
Huyen
2011/09/28 18:55:48
Done.
|
| systeminstall.UnInstall(); |
| } else { |
| - ChromeMiniInstaller userinstall(kUserInstall, |
| - cmd->HasSwitch(installer::switches::kChromeFrame)); |
| + ChromeMiniInstaller userinstall(false, false); |
| userinstall.UnInstall(); |
| - ChromeMiniInstaller systeminstall(kSystemInstall, |
| - cmd->HasSwitch(installer::switches::kChromeFrame)); |
| + ChromeMiniInstaller systeminstall(true,false); |
|
kkania
2011/09/28 18:02:43
space
|
| systeminstall.UnInstall(); |
| } |
| } |
| @@ -53,10 +50,8 @@ class MiniInstallTest : public testing::Test { |
| // Create a few differently configured installers that are used in |
| // the tests, for convenience. |
| - user_inst_.reset(new ChromeMiniInstaller(kUserInstall, |
| - chrome_frame_)); |
| - sys_inst_.reset(new ChromeMiniInstaller(kSystemInstall, |
| - chrome_frame_)); |
| + user_inst_.reset(new ChromeMiniInstaller(false, chrome_frame_)); |
| + sys_inst_.reset(new ChromeMiniInstaller(true, chrome_frame_)); |
| sys_inst_->SetBuildUnderTest(build); |
| user_inst_->SetBuildUnderTest(build); |
| } |
| @@ -175,6 +170,16 @@ TEST_F(MiniInstallTest, |
| if (!chrome_frame_) |
| user_inst_->OverInstall(); |
| } |
| + |
| +TEST_F(MiniInstallTest, |
| + InstallChromeUsingMultiInstallUser) { |
| + user_inst_->InstallUsingMultiInstall(); |
| +} |
| + |
| +TEST_F(MiniInstallTest, |
| + InstallChromeUsingMultiInstallSys) { |
| + sys_inst_->InstallUsingMultiInstall(); |
| +} |
| #endif |
| TEST_F(MiniInstallTest, InstallMiniInstallerSys) { |