Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1510)

Unified Diff: chrome/browser/web_applications/web_app_win.cc

Issue 1289333005: Change shortcut install location to non-subdir. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: git cl format Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/chrome_browser_main_win.cc ('k') | chrome/installer/setup/install.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/web_applications/web_app_win.cc
diff --git a/chrome/browser/web_applications/web_app_win.cc b/chrome/browser/web_applications/web_app_win.cc
index a5078b016477452ef58ba56471e5f83a6011dfd7..572fd2827e7f31db21d7b4e828658c0a024a5420 100644
--- a/chrome/browser/web_applications/web_app_win.cc
+++ b/chrome/browser/web_applications/web_app_win.cc
@@ -565,28 +565,20 @@ std::vector<base::FilePath> GetShortcutPaths(
bool use_this_location;
ShellUtil::ShortcutLocation location_id;
} locations[] = {
- {
- creation_locations.on_desktop,
- ShellUtil::SHORTCUT_LOCATION_DESKTOP
- }, {
- creation_locations.applications_menu_location == APP_MENU_LOCATION_ROOT,
- ShellUtil::SHORTCUT_LOCATION_START_MENU_ROOT
- }, {
- creation_locations.applications_menu_location ==
- APP_MENU_LOCATION_SUBDIR_CHROME,
- ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_DIR
- }, {
- creation_locations.applications_menu_location ==
- APP_MENU_LOCATION_SUBDIR_CHROMEAPPS,
- ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_APPS_DIR
- }, {
- // For Win7+, |in_quick_launch_bar| indicates that we are pinning to
- // taskbar. This needs to be handled by callers.
- creation_locations.in_quick_launch_bar &&
- base::win::CanPinShortcutToTaskbar(),
- ShellUtil::SHORTCUT_LOCATION_QUICK_LAUNCH
- }
- };
+ {creation_locations.on_desktop, ShellUtil::SHORTCUT_LOCATION_DESKTOP},
grt (UTC plus 2) 2015/10/22 17:31:11 "git cl format" is very useful, but sometimes does
bcwhite 2015/10/23 17:22:14 Done.
+ {creation_locations.applications_menu_location == APP_MENU_LOCATION_ROOT,
+ ShellUtil::SHORTCUT_LOCATION_START_MENU_ROOT},
+ {creation_locations.applications_menu_location ==
+ APP_MENU_LOCATION_SUBDIR_CHROME,
+ ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_DIR_DEPRECATED},
+ {creation_locations.applications_menu_location ==
+ APP_MENU_LOCATION_SUBDIR_CHROMEAPPS,
+ ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_APPS_DIR},
+ {// For Win7+, |in_quick_launch_bar| indicates that we are pinning to
+ // taskbar. This needs to be handled by callers.
+ creation_locations.in_quick_launch_bar &&
+ base::win::CanPinShortcutToTaskbar(),
+ ShellUtil::SHORTCUT_LOCATION_QUICK_LAUNCH}};
BrowserDistribution* dist = BrowserDistribution::GetDistribution();
// Populate shortcut_paths.
« no previous file with comments | « chrome/browser/chrome_browser_main_win.cc ('k') | chrome/installer/setup/install.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698