Polish UI of bookmark app creation.

chrome/browser/ui/toolbar/wrench_menu_model.cc

Index: chrome/browser/ui/toolbar/wrench_menu_model.cc
diff --git a/chrome/browser/ui/toolbar/wrench_menu_model.cc b/chrome/browser/ui/toolbar/wrench_menu_model.cc
index 010f4d2de80d9361cc2f10852e6f29ceec0d25cb..3684113ec394f1544b838fc2ae6423a960b167a2 100644
--- a/chrome/browser/ui/toolbar/wrench_menu_model.cc
+++ b/chrome/browser/ui/toolbar/wrench_menu_model.cc
@@ -240,7 +240,18 @@ void ToolsMenuModel::Build(Browser* browser) {
 #endif
 
   if (extensions::util::IsNewBookmarkAppsEnabled()) {
-    AddItemWithStringId(IDC_CREATE_HOSTED_APP, IDS_CREATE_HOSTED_APP);
+#if defined(OS_MACOSX)
+    int string_id = IDS_ADD_TO_DOCK;
+#elif defined(OS_WIN)
+    int string_id = IDS_ADD_TO_TASKBAR;
+#else
+    int string_id = IDS_ADD_TO_DESKTOP;

[sashab, 2015/02/06 15:45:22]

Not needed if they all use the same string ID.

[benwells, 2015/02/06 15:52:17]

Yes I understand. As described earlier I think it is clearer like this (in
particular, for translators). Let's see what owners think, maybe they'll have an
opinion...

+#endif
+#if defined(USE_ASH)
+    if (browser->host_desktop_type() == chrome::HOST_DESKTOP_TYPE_ASH)
+      string_id = IDS_ADD_TO_SHELF;
+#endif
+    AddItemWithStringId(IDC_CREATE_HOSTED_APP, string_id);
     AddSeparator(ui::NORMAL_SEPARATOR);
   } else if (show_create_shortcuts) {
     AddItemWithStringId(IDC_CREATE_SHORTCUTS, IDS_CREATE_SHORTCUTS);