Index: chrome/browser/ui/views/create_application_shortcut_view.cc |
diff --git a/chrome/browser/ui/views/create_application_shortcut_view.cc b/chrome/browser/ui/views/create_application_shortcut_view.cc |
index 3f4feb8ca092f0116b7f1e33ccc4f668a9c3edbe..908371551edd4c4ca6ab164b9960b85b59264f65 100644 |
--- a/chrome/browser/ui/views/create_application_shortcut_view.cc |
+++ b/chrome/browser/ui/views/create_application_shortcut_view.cc |
@@ -376,10 +376,12 @@ bool CreateApplicationShortcutView::Accept() { |
ShellIntegration::ShortcutLocations creation_locations; |
creation_locations.on_desktop = desktop_check_box_->checked(); |
- creation_locations.in_applications_menu = menu_check_box_ == NULL ? false : |
- menu_check_box_->checked(); |
- creation_locations.applications_menu_subdir = shortcut_menu_subdir_; |
- |
+ if (menu_check_box_ != NULL && menu_check_box_->checked()) { |
Matt Giuca
2013/05/15 05:19:45
You will need to make this change in chrome/browse
|
+ if (create_in_chrome_apps_subdir_) |
+ creation_locations.in_applications_menu_chrome_apps_subdir = true; |
+ else |
+ creation_locations.in_applications_menu = true; |
+ } |
#if defined(OS_WIN) |
creation_locations.in_quick_launch_bar = quick_launch_check_box_ == NULL ? |
NULL : quick_launch_check_box_->checked(); |
@@ -433,8 +435,8 @@ CreateUrlApplicationShortcutView::CreateUrlApplicationShortcutView( |
FetchIcon(); |
} |
- // NOTE: Leave shortcut_menu_subdir_ blank to create URL app shortcuts in the |
- // top-level menu. |
+ // Create URL app shortcuts in the top-level menu. |
+ create_in_chrome_apps_subdir_ = false; |
Matt Giuca
2013/05/15 05:19:45
I'm glad this comment came in handy :)
|
InitControls(); |
} |
@@ -516,7 +518,7 @@ CreateChromeApplicationShortcutView::CreateChromeApplicationShortcutView( |
shortcut_info_.description = UTF8ToUTF16(app->description()); |
// Place Chrome app shortcuts in the "Chrome Apps" submenu. |
- shortcut_menu_subdir_ = web_app::GetAppShortcutsSubdirName(); |
+ create_in_chrome_apps_subdir_ = true; |
InitControls(); |