From a1b29082fe381f0d8db227f2e8e967baaae245bf Mon Sep 17 00:00:00 2001 From: gab Date: Mon, 27 Jul 2015 09:34:26 -0700 Subject: No dual_mode on Win10+ shortcuts. BUG=501166 Review URL: https://codereview.chromium.org/1252513002 Cr-Commit-Position: refs/heads/master@{#340479} --- chrome/installer/setup/install.cc | 7 ++++--- chrome/installer/setup/install_unittest.cc | 4 +++- chrome/installer/setup/setup_main.cc | 3 ++- 3 files changed, 9 insertions(+), 5 deletions(-) (limited to 'chrome/installer/setup') diff --git a/chrome/installer/setup/install.cc b/chrome/installer/setup/install.cc index 37a1417..00a74af 100644 --- a/chrome/installer/setup/install.cc +++ b/chrome/installer/setup/install.cc @@ -429,9 +429,10 @@ void CreateOrUpdateShortcuts( ShellUtil::ShortcutProperties start_menu_properties(base_properties); // IMPORTANT: Only the default (no arguments and default browserappid) browser - // shortcut in the Start menu (Start screen on Win8+) should be made dual - // mode. - start_menu_properties.set_dual_mode(true); + // shortcut in the Start menu (Start screen on Win8+) should be considered for + // dual mode. + if (InstallUtil::ShouldInstallMetroProperties()) + start_menu_properties.set_dual_mode(true); if (!do_not_create_taskbar_shortcut && (shortcut_operation == ShellUtil::SHELL_SHORTCUT_CREATE_ALWAYS || shortcut_operation == diff --git a/chrome/installer/setup/install_unittest.cc b/chrome/installer/setup/install_unittest.cc index f9256a0..8c48202 100644 --- a/chrome/installer/setup/install_unittest.cc +++ b/chrome/installer/setup/install_unittest.cc @@ -21,6 +21,7 @@ #include "chrome/installer/setup/install_worker.h" #include "chrome/installer/setup/setup_constants.h" #include "chrome/installer/util/browser_distribution.h" +#include "chrome/installer/util/install_util.h" #include "chrome/installer/util/installer_state.h" #include "chrome/installer/util/master_preferences.h" #include "chrome/installer/util/master_preferences_constants.h" @@ -87,7 +88,8 @@ class InstallShortcutTest : public testing::Test { expected_properties_.set_description(chrome_properties.description); expected_properties_.set_dual_mode(false); expected_start_menu_properties_ = expected_properties_; - expected_start_menu_properties_.set_dual_mode(true); + expected_start_menu_properties_.set_dual_mode( + InstallUtil::ShouldInstallMetroProperties()); prefs_.reset(GetFakeMasterPrefs(false, false, false)); diff --git a/chrome/installer/setup/setup_main.cc b/chrome/installer/setup/setup_main.cc index 6b419c7..aac2781 100644 --- a/chrome/installer/setup/setup_main.cc +++ b/chrome/installer/setup/setup_main.cc @@ -880,7 +880,8 @@ installer::InstallStatus RegisterDevChrome( // Create the Start menu shortcut and pin it to the taskbar. ShellUtil::ShortcutProperties shortcut_properties(ShellUtil::CURRENT_USER); chrome.AddDefaultShortcutProperties(chrome_exe, &shortcut_properties); - shortcut_properties.set_dual_mode(true); + if (InstallUtil::ShouldInstallMetroProperties()) + shortcut_properties.set_dual_mode(true); shortcut_properties.set_pin_to_taskbar(true); ShellUtil::CreateOrUpdateShortcut( ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_DIR, chrome_dist, -- cgit v1.1