Remove base/win/metro.{cc|h} and some associated code

chrome/browser/shell_integration_win_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/shell_integration.h"
 
 #include <stddef.h>
 
 #include <vector>
 
@@ skipping 18 matching lines @@
 
 namespace {
 
 struct ShortcutTestObject {
   base::FilePath path;
   base::win::ShortcutProperties properties;
 };
 
 class ShellIntegrationWinMigrateShortcutTest : public testing::Test {
  protected:
-  ShellIntegrationWinMigrateShortcutTest()
-      : desired_dual_mode_for_os_version(
-            InstallUtil::ShouldInstallMetroProperties()) {}
+  ShellIntegrationWinMigrateShortcutTest() {}
 
   void SetUp() override {
     ASSERT_TRUE(temp_dir_.CreateUniqueTempDir());
 
     // A path to a random target.
     base::CreateTemporaryFileInDir(temp_dir_.path(), &other_target_);
 
     // This doesn't need to actually have a base name of "chrome.exe".
     base::CreateTemporaryFileInDir(temp_dir_.path(), &chrome_exe_);
 
@@ skipping 205 matching lines @@
 
   // An example extension id of an example app.
   base::string16 extension_id_;
 
   // The app id of the example app for the default profile and user data dir.
   base::string16 extension_app_id_;
 
   // The app id of the example app for the non-default profile.
   base::string16 non_default_profile_extension_app_id_;
 
-  // True if the dual mode property should be set for the default chrome
-  // shortcut on the current OS version.
-  const bool desired_dual_mode_for_os_version;
-
  private:
   DISALLOW_COPY_AND_ASSIGN(ShellIntegrationWinMigrateShortcutTest);
 };
 
 }  // namespace
 
 // Test migration when not checking for dual mode.
 TEST_F(ShellIntegrationWinMigrateShortcutTest, DontCheckDualMode) {
   if (base::win::GetVersion() < base::win::VERSION_WIN7)
     return;
@@ skipping 34 matching lines @@
                 chrome_exe_, temp_dir_.path(), false));
 }
 
 // Test migration when also checking for dual mode.
 TEST_F(ShellIntegrationWinMigrateShortcutTest, CheckDualMode) {
   if (base::win::GetVersion() < base::win::VERSION_WIN7)
     return;
 
   // 9 shortcuts should have their app id updated below.
 
-  // If |desired_dual_mode_for_os_version| is true: shortcut 2 and 13 should
-  //   also be migrated to dual_mode for a total of 11 shortcuts migrated.
-  // If |desired_dual_mode_for_os_version| is false: shortcut 11 should
-  //   be migrate away from dual_mode for a total of 10 shortcuts migrated.
-  EXPECT_EQ(desired_dual_mode_for_os_version ? 11 : 10,
-            ShellIntegration::MigrateShortcutsInPathInternal(
-                chrome_exe_, temp_dir_.path(), true));
+  // shortcut 11 should be migrate away from dual_mode for a total of 10
+  // shortcuts migrated.
+  EXPECT_EQ(10, ShellIntegration::MigrateShortcutsInPathInternal(
+                    chrome_exe_, temp_dir_.path(), true));
 
   // Shortcut 1, 3, 4, 5, 6, 7, 8, 9, and 10 should have had both their app_id
   // fixed and shortcut 1, 2, 3, 4, and 5 should also have had their dual_mode
   // property added if it is desired (or 11 should have had it removed if it is
   // not).
   shortcuts_[1].properties.set_app_id(chrome_app_id_);
   shortcuts_[3].properties.set_app_id(chrome_app_id_);
   shortcuts_[4].properties.set_app_id(chrome_app_id_);
   shortcuts_[5].properties.set_app_id(chrome_app_id_);
   shortcuts_[6].properties.set_app_id(non_default_profile_chrome_app_id_);
   shortcuts_[7].properties.set_app_id(non_default_user_data_dir_chrome_app_id_);
   shortcuts_[8].properties.set_app_id(
       non_default_user_data_dir_and_profile_chrome_app_id_);
   shortcuts_[9].properties.set_app_id(extension_app_id_);
   shortcuts_[10].properties.set_app_id(non_default_profile_extension_app_id_);
 
   // Explicitly flag the expected dual_mode properties.
   shortcuts_[0].properties.set_dual_mode(false);
-  if (desired_dual_mode_for_os_version) {
-    shortcuts_[1].properties.set_dual_mode(true);
-    shortcuts_[2].properties.set_dual_mode(true);
-    shortcuts_[3].properties.set_dual_mode(true);
-    shortcuts_[4].properties.set_dual_mode(true);
-    shortcuts_[5].properties.set_dual_mode(true);
-    shortcuts_[12].properties.set_dual_mode(true);
-  } else {
-    shortcuts_[11].properties.set_dual_mode(false);
-  }
+  shortcuts_[11].properties.set_dual_mode(false);
   shortcuts_[6].properties.set_dual_mode(false);
   shortcuts_[7].properties.set_dual_mode(false);
   shortcuts_[8].properties.set_dual_mode(false);
   shortcuts_[9].properties.set_dual_mode(false);
   shortcuts_[10].properties.set_dual_mode(false);
 
   for (size_t i = 0; i < shortcuts_.size(); ++i) {
     SCOPED_TRACE(i);
     base::win::ValidateShortcut(shortcuts_[i].path, shortcuts_[i].properties);
   }
@@ skipping 52 matching lines @@
                 default_profile_path));
 
   // Non-default profile path should get chrome::kBrowserAppID + AppList joined
   // with profile info.
   base::FilePath profile_path(FILE_PATH_LITERAL("root"));
   profile_path = profile_path.Append(FILE_PATH_LITERAL("udd"));
   profile_path = profile_path.Append(FILE_PATH_LITERAL("User Data - Test"));
   EXPECT_EQ(base_app_id + L".udd.UserDataTest",
             ShellIntegration::GetAppListAppModelIdForProfile(profile_path));
 }