Index: chrome/installer/setup/install_unittest.cc |
diff --git a/chrome/installer/setup/install_unittest.cc b/chrome/installer/setup/install_unittest.cc |
index 0cb3e8fd26f4e2a708c1a75976d55e7c4c52a7db..6f8b9c09c152e5ad2329929a30ec979c23f263ed 100644 |
--- a/chrome/installer/setup/install_unittest.cc |
+++ b/chrome/installer/setup/install_unittest.cc |
@@ -125,21 +125,18 @@ |
fake_user_desktop_.path().Append(shortcut_name); |
user_quick_launch_shortcut_ = |
fake_user_quick_launch_.path().Append(shortcut_name); |
- user_start_menu_shortcut_ = fake_start_menu_.path().Append(shortcut_name); |
- user_start_menu_subdir_shortcut_ = |
- fake_start_menu_.path() |
- .Append(dist_->GetStartMenuShortcutSubfolder( |
+ user_start_menu_shortcut_ = |
+ fake_start_menu_.path().Append( |
+ dist_->GetStartMenuShortcutSubfolder( |
BrowserDistribution::SUBFOLDER_CHROME)) |
- .Append(shortcut_name); |
+ .Append(shortcut_name); |
system_desktop_shortcut_ = |
fake_common_desktop_.path().Append(shortcut_name); |
system_start_menu_shortcut_ = |
- fake_common_start_menu_.path().Append(shortcut_name); |
- system_start_menu_subdir_shortcut_ = |
- fake_common_start_menu_.path() |
- .Append(dist_->GetStartMenuShortcutSubfolder( |
+ fake_common_start_menu_.path().Append( |
+ dist_->GetStartMenuShortcutSubfolder( |
BrowserDistribution::SUBFOLDER_CHROME)) |
- .Append(shortcut_name); |
+ .Append(shortcut_name); |
user_alternate_desktop_shortcut_ = |
fake_user_desktop_.path().Append(alternate_shortcut_name); |
} |
@@ -148,9 +145,7 @@ |
// Try to unpin potentially pinned shortcuts (although pinning isn't tested, |
// the call itself might still have pinned the Start Menu shortcuts). |
base::win::UnpinShortcutFromTaskbar(user_start_menu_shortcut_); |
- base::win::UnpinShortcutFromTaskbar(user_start_menu_subdir_shortcut_); |
base::win::UnpinShortcutFromTaskbar(system_start_menu_shortcut_); |
- base::win::UnpinShortcutFromTaskbar(system_start_menu_subdir_shortcut_); |
CoUninitialize(); |
} |
@@ -205,10 +200,8 @@ |
base::FilePath user_desktop_shortcut_; |
base::FilePath user_quick_launch_shortcut_; |
base::FilePath user_start_menu_shortcut_; |
- base::FilePath user_start_menu_subdir_shortcut_; |
base::FilePath system_desktop_shortcut_; |
base::FilePath system_start_menu_shortcut_; |
- base::FilePath system_start_menu_subdir_shortcut_; |
base::FilePath user_alternate_desktop_shortcut_; |
}; |
@@ -382,69 +375,6 @@ |
ASSERT_FALSE(base::PathExists(user_start_menu_shortcut_)); |
} |
-class MigrateShortcutTest : public InstallShortcutTest, |
- public testing::WithParamInterface< |
- testing::tuple< |
- installer::InstallShortcutOperation, |
- installer::InstallShortcutLevel>> { |
- public: |
- MigrateShortcutTest() : shortcut_operation_(testing::get<0>(GetParam())), |
- shortcut_level_(testing::get<1>(GetParam())) {} |
- |
- protected: |
- const installer::InstallShortcutOperation shortcut_operation_; |
- const installer::InstallShortcutLevel shortcut_level_; |
- |
- private: |
- DISALLOW_COPY_AND_ASSIGN(MigrateShortcutTest); |
-}; |
- |
-TEST_P(MigrateShortcutTest, MigrateAwayFromDeprecatedStartMenuTest) { |
- base::win::ShortcutProperties dummy_properties; |
- base::FilePath dummy_target; |
- ASSERT_TRUE(base::CreateTemporaryFileInDir(temp_dir_.path(), &dummy_target)); |
- dummy_properties.set_target(expected_properties_.target); |
- dummy_properties.set_working_dir(fake_user_desktop_.path()); |
- dummy_properties.set_arguments(L"--dummy --args"); |
- dummy_properties.set_app_id(L"El.Dummiest"); |
- |
- base::FilePath start_menu_shortcut; |
- base::FilePath start_menu_subdir_shortcut; |
- if (shortcut_level_ == installer::CURRENT_USER) { |
- start_menu_shortcut = user_start_menu_shortcut_; |
- start_menu_subdir_shortcut = user_start_menu_subdir_shortcut_; |
- } else { |
- start_menu_shortcut = system_start_menu_shortcut_; |
- start_menu_subdir_shortcut = system_start_menu_subdir_shortcut_; |
- } |
- |
- ASSERT_TRUE(base::CreateDirectory(start_menu_subdir_shortcut.DirName())); |
- ASSERT_FALSE(base::PathExists(start_menu_subdir_shortcut)); |
- ASSERT_TRUE(base::win::CreateOrUpdateShortcutLink( |
- start_menu_subdir_shortcut, dummy_properties, |
- base::win::SHORTCUT_CREATE_ALWAYS)); |
- ASSERT_TRUE(base::PathExists(start_menu_subdir_shortcut)); |
- ASSERT_FALSE(base::PathExists(start_menu_shortcut)); |
- |
- installer::CreateOrUpdateShortcuts(chrome_exe_, *product_, *prefs_, |
- shortcut_level_, shortcut_operation_); |
- ASSERT_FALSE(base::PathExists(start_menu_subdir_shortcut)); |
- ASSERT_TRUE(base::PathExists(start_menu_shortcut)); |
-} |
- |
-// Verify that any installer operation for any installation level triggers |
-// the migration from sub-folder to root of start-menu. |
-INSTANTIATE_TEST_CASE_P( |
- MigrateShortcutTests, MigrateShortcutTest, |
- testing::Combine( |
- testing::Values( |
- installer::INSTALL_SHORTCUT_REPLACE_EXISTING, |
- installer::INSTALL_SHORTCUT_CREATE_EACH_IF_NO_SYSTEM_LEVEL, |
- installer::INSTALL_SHORTCUT_CREATE_ALL), |
- testing::Values( |
- installer::CURRENT_USER, |
- installer::ALL_USERS))); |
- |
TEST_F(InstallShortcutTest, CreateIfNoSystemLevelAllSystemShortcutsExist) { |
base::win::ShortcutProperties dummy_properties; |
base::FilePath dummy_target; |