reland "views: change WrenchMenu to use each model's command ID's when creating MenuItemView's"

chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h

Index: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
index 62d9a078f5d06e99b2b35e5ca29a28c0190d94ec..691b551a4c37a1a03df22e93bca050f2e5c2624d 100644
--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
+++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.h
@@ -39,6 +39,11 @@ class AcceleratorProvider;
 class RecentTabsSubMenuModel : public ui::SimpleMenuModel,
                                public ui::SimpleMenuModel::Delegate {
  public:
+  // Command Id for recently closed items header or disabled item to which the
+  // accelerator string will be appended.
+  static const int kRecentlyClosedHeaderCommandId;
+  static const int kDisabledRecentlyClosedHeaderCommandId;
+
   // If |associator| is NULL, default associator for |browser|'s profile will
   // be used.  Testing may require a specific |associator|.
   RecentTabsSubMenuModel(ui::AcceleratorProvider* accelerator_provider,
@@ -60,14 +65,6 @@ class RecentTabsSubMenuModel : public ui::SimpleMenuModel,
                                     std::string* url,
                                     string16* title) const;
 
-  // Command Id for recently closed items header or disabled item to which the
-  // accelerator string will be appended.
-  static const int kRecentlyClosedHeaderCommandId;
-  static const int kDisabledRecentlyClosedHeaderCommandId;
-
-  // Command Id for other devices section headers, using the device name.
-  static const int kDeviceNameCommandId;
-
  private:
   struct TabNavigationItem;
   typedef std::vector<TabNavigationItem> TabNavigationItems;