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

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: added menu_root to cleanup code 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
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..e0151d807b6c5ef199f6ac7ac3e5b76629ae8933 100644
--- a/chrome/browser/web_applications/web_app_win.cc
+++ b/chrome/browser/web_applications/web_app_win.cc
@@ -574,7 +574,7 @@ std::vector<base::FilePath> GetShortcutPaths(
}, {
creation_locations.applications_menu_location ==
APP_MENU_LOCATION_SUBDIR_CHROME,
gab 2015/10/20 18:25:59 Should also remove APP_MENU_LOCATION_SUBDIR_CHROME
bcwhite 2015/10/20 19:03:12 app_list_service_win.cc:300 (CreateShortcut) This
grt (UTC plus 2) 2015/10/22 17:31:11 Are you guys talking about the same thing? The "ap
gab 2015/10/22 21:34:45 I thought the App Launcher had been phased out, ha
bcwhite 2015/10/23 17:22:14 Okay, marked deprecated.
gab 2015/10/23 18:00:23 Actually not convinced it should be deprecated, ra
- ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_DIR
+ ShellUtil::SHORTCUT_LOCATION_START_MENU_CHROME_DIR_DEPRECATED
}, {
creation_locations.applications_menu_location ==
APP_MENU_LOCATION_SUBDIR_CHROMEAPPS,

Powered by Google App Engine
This is Rietveld 408576698