diff options
| author | grt <grt@chromium.org> | 2015-06-22 21:52:21 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-06-23 04:53:01 +0000 |
| commit | fdeaa0e1d9ca6166b27e5f8ac19bba3a5ee9fcb2 (patch) | |
| tree | 166cb5dcd8b120f151ab56d9cad72ce1f1756920 /chrome/installer/setup | |
| parent | 69c24ff2eadf87a6ebd233ad79643b41d9e7e1e3 (diff) | |
| download | chromium_src-fdeaa0e1d9ca6166b27e5f8ac19bba3a5ee9fcb2.zip chromium_src-fdeaa0e1d9ca6166b27e5f8ac19bba3a5ee9fcb2.tar.gz chromium_src-fdeaa0e1d9ca6166b27e5f8ac19bba3a5ee9fcb2.tar.bz2 | |
Pin shortcuts via shell verbs rather than ShellExecuteEx.
BUG=498383
TEST=Chrome's icon is pinned to the taskbar upon install on Win7, 8, 8.1, and 10.
TBR=sky@chromium.org
Review URL: https://codereview.chromium.org/1193363003
Cr-Commit-Position: refs/heads/master@{#335630}
Diffstat (limited to 'chrome/installer/setup')
| -rw-r--r-- | chrome/installer/setup/install_unittest.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/chrome/installer/setup/install_unittest.cc b/chrome/installer/setup/install_unittest.cc index d011f7e..2839cd8 100644 --- a/chrome/installer/setup/install_unittest.cc +++ b/chrome/installer/setup/install_unittest.cc @@ -142,10 +142,8 @@ class InstallShortcutTest : public testing::Test { void TearDown() override { // Try to unpin potentially pinned shortcuts (although pinning isn't tested, // the call itself might still have pinned the Start Menu shortcuts). - base::win::TaskbarUnpinShortcutLink( - user_start_menu_shortcut_.value().c_str()); - base::win::TaskbarUnpinShortcutLink( - system_start_menu_shortcut_.value().c_str()); + base::win::TaskbarUnpinShortcutLink(user_start_menu_shortcut_); + base::win::TaskbarUnpinShortcutLink(system_start_menu_shortcut_); CoUninitialize(); } |