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

chrome/browser/ui/toolbar/wrench_menu_model.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_UI_TOOLBAR_WRENCH_MENU_MODEL_H_
 #define CHROME_BROWSER_UI_TOOLBAR_WRENCH_MENU_MODEL_H_
 
 #include "base/files/file_path.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
@@ skipping 61 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(ToolsMenuModel);
 };
 
 // A menu model that builds the contents of the wrench menu.
 class WrenchMenuModel : public ui::SimpleMenuModel,
                         public ui::SimpleMenuModel::Delegate,
                         public ui::ButtonMenuItemModel::Delegate,
                         public TabStripModelObserver,
                         public content::NotificationObserver {
  public:
+  // Range of command ID's to use for the items representing bookmarks in the
+  // bookmark menu, must not overlap with that for recent tabs submenu.
+  static const int kMinBookmarkCommandId = 1;
+  static const int kMaxBookmarkCommandId = 1000;
+
+  // Range of command ID's to use for the items in the recent tabs submenu, must
+  // not overlap with that for bookmarks.
+  static const int kMinRecentTabsCommandId = 1001;
+  static const int kMaxRecentTabsCommandId = 1200;
+
   // TODO: remove |is_new_menu|.
   WrenchMenuModel(ui::AcceleratorProvider* provider,
                   Browser* browser,
                   bool is_new_menu);
   virtual ~WrenchMenuModel();
 
   // Overridden for ButtonMenuItemModel::Delegate:
   virtual bool DoesCommandIdDismissMenu(int command_id) const OVERRIDE;
 
   // Overridden for both ButtonMenuItemModel::Delegate and SimpleMenuModel:
@@ skipping 81 matching lines @@
   Browser* browser_;  // weak
   TabStripModel* tab_strip_model_; // weak
 
   scoped_ptr<content::HostZoomMap::Subscription> zoom_subscription_;
   content::NotificationRegistrar registrar_;
 
   DISALLOW_COPY_AND_ASSIGN(WrenchMenuModel);
 };
 
 #endif  // CHROME_BROWSER_UI_TOOLBAR_WRENCH_MENU_MODEL_H_