Moved some Page menu elements to App menu (they fit there much more); these a...

chrome/browser/views/toolbar_view.cc

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/views/toolbar_view.h"
 
 #include <string>
 
 #include "base/logging.h"
 #include "base/path_service.h"
@@ skipping 469 matching lines @@
     out->cy = ps.cy + 2 * kControlVertOffsetLocationOnly;
   }
 }
 
 void BrowserToolbarView::RunPageMenu(const CPoint& pt, HWND hwnd) {
   Menu::AnchorPoint anchor = Menu::TOPRIGHT;
   if (UILayoutIsRightToLeft())
     anchor = Menu::TOPLEFT;
 
   Menu menu(this, anchor, hwnd);
-  menu.AppendMenuItemWithLabel(IDC_NEWTAB, l10n_util::GetString(IDS_NEWTAB));
-  menu.AppendMenuItemWithLabel(IDC_NEWWINDOW,
-                               l10n_util::GetString(IDS_NEWWINDOW));
-  menu.AppendMenuItemWithLabel(IDC_GOOFFTHERECORD,
-                               l10n_util::GetString(IDS_GOOFFTHERECORD));
-  menu.AppendSeparator();
   // The install menu may be dynamically generated with a contextual label.
   // See browser_commands.cc.
   menu.AppendMenuItemWithLabel(IDC_CREATE_SHORTCUT,
       l10n_util::GetString(IDS_DEFAULT_INSTALL_SITE_LABEL));
   menu.AppendSeparator();
   menu.AppendMenuItemWithLabel(IDC_CUT, l10n_util::GetString(IDS_CUT));
   menu.AppendMenuItemWithLabel(IDC_COPY, l10n_util::GetString(IDS_COPY));
   menu.AppendMenuItemWithLabel(IDC_PASTE, l10n_util::GetString(IDS_PASTE));
   menu.AppendSeparator();
 
@@ skipping 49 matching lines @@
                                l10n_util::GetString(IDS_COMMANDS_REPORTBUG));
   menu.RunMenuAt(pt.x, pt.y);
 }
 
 void BrowserToolbarView::RunAppMenu(const CPoint& pt, HWND hwnd) {
   Menu::AnchorPoint anchor = Menu::TOPRIGHT;
   if (UILayoutIsRightToLeft())
     anchor = Menu::TOPLEFT;
 
   Menu menu(this, anchor, hwnd);
+  menu.AppendMenuItemWithLabel(IDC_NEWTAB, l10n_util::GetString(IDS_NEWTAB));
+  menu.AppendMenuItemWithLabel(IDC_NEWWINDOW,
+                               l10n_util::GetString(IDS_NEWWINDOW));
+  menu.AppendMenuItemWithLabel(IDC_GOOFFTHERECORD,
+                               l10n_util::GetString(IDS_GOOFFTHERECORD));
+  menu.AppendSeparator();
   menu.AppendMenuItemWithLabel(IDC_SHOW_BOOKMARKS_BAR,
                                l10n_util::GetString(IDS_SHOW_BOOKMARK_BAR));
   menu.AppendSeparator();
   menu.AppendMenuItemWithLabel(IDC_SHOW_HISTORY,
                                l10n_util::GetString(IDS_SHOW_HISTORY));
   menu.AppendMenuItemWithLabel(IDC_SHOW_DOWNLOADS,
                                l10n_util::GetString(IDS_SHOW_DOWNLOADS));
   menu.AppendSeparator();
   menu.AppendMenuItemWithLabel(IDC_CLEAR_BROWSING_DATA,
                                l10n_util::GetString(IDS_CLEAR_BROWSING_DATA));
@@ skipping 138 matching lines @@
       *accel = ChromeViews::Accelerator(L'C', false, true, false);
       return true;
     case IDC_PASTE:
       *accel = ChromeViews::Accelerator(L'V', false, true, false);
       return true;
   }
   // Else, we retrieve the accelerator information from the frame.
   return GetViewContainer()->GetAccelerator(id, accel);
 }