Makes bookmark menu lazily create menus and removes limits

chrome/browser/ui/views/bookmarks/bookmark_menu_delegate.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_VIEWS_BOOKMARKS_BOOKMARK_MENU_DELEGATE_H_
 #define CHROME_BROWSER_UI_VIEWS_BOOKMARKS_BOOKMARK_MENU_DELEGATE_H_
 
 #include <map>
 #include <set>
 
@@ skipping 34 matching lines @@
     // than then bookmark bar folder). This only makes sense when showing the
     // contents of the bookmark bar folder.
     SHOW_PERMANENT_FOLDERS,
 
     // Don't show any additional folders.
     HIDE_PERMANENT_FOLDERS
   };
 
   BookmarkMenuDelegate(Browser* browser,
                        content::PageNavigator* navigator,
-                       views::Widget* parent,
-                       int first_menu_id,
-                       int max_menu_id);
+                       views::Widget* parent);
   ~BookmarkMenuDelegate() override;
 
   // Creates the menus from the model.
   void Init(views::MenuDelegate* real_delegate,
             views::MenuItemView* parent,
             const bookmarks::BookmarkNode* node,
             int start_child_index,
             ShowOptions show_options,
             BookmarkLaunchLocation location);
 
@@ skipping 44 matching lines @@
                     views::MenuDelegate::DropPosition position,
                     const ui::DropTargetEvent& event);
   bool ShowContextMenu(views::MenuItemView* source,
                        int id,
                        const gfx::Point& p,
                        ui::MenuSourceType source_type);
   bool CanDrag(views::MenuItemView* menu);
   void WriteDragData(views::MenuItemView* sender, ui::OSExchangeData* data);
   int GetDragOperations(views::MenuItemView* sender);
   int GetMaxWidthForMenu(views::MenuItemView* menu);
+  void WillShowMenu(views::MenuItemView* menu);
 
   // BookmarkModelObserver methods.
   void BookmarkModelChanged() override;
   void BookmarkNodeFaviconChanged(bookmarks::BookmarkModel* model,
                                   const bookmarks::BookmarkNode* node) override;
 
   // BookmarkContextMenu::Observer methods.
   void WillRemoveBookmarks(
       const std::vector<const bookmarks::BookmarkNode*>& bookmarks) override;
   void DidRemoveBookmarks() override;
 
  private:
+  friend class BookmarkMenuDelegateTest;
+
   typedef std::map<int, const bookmarks::BookmarkNode*> MenuIDToNodeMap;
   typedef std::map<const bookmarks::BookmarkNode*, views::MenuItemView*>
       NodeToMenuMap;
 
   // Creates a menu. This uses BuildMenu() to recursively populate the menu.
   views::MenuItemView* CreateMenu(const bookmarks::BookmarkNode* parent,
                                   int start_child_index,
                                   ShowOptions show_options);
 
   // Invokes BuildMenuForPermanentNode() for the permanent nodes (excluding
   // 'other bookmarks' folder).
-  void BuildMenusForPermanentNodes(views::MenuItemView* menu,
-                                   int* next_menu_id);
+  void BuildMenusForPermanentNodes(views::MenuItemView* menu);
 
   // If |node| has children a new menu is created and added to |menu| to
   // represent it. If |node| is not empty and |added_separator| is false, a
   // separator is added before the new menu items and |added_separator| is set
   // to true.
   void BuildMenuForPermanentNode(const bookmarks::BookmarkNode* node,
                                  int icon_resource_id,
                                  views::MenuItemView* menu,
-                                 int* next_menu_id,
                                  bool* added_separator);
 
-  void BuildMenuForManagedNode(views::MenuItemView* menu, int* next_menu_id);
-  void BuildMenuForSupervisedNode(views::MenuItemView* menu, int* next_menu_id);
+  void BuildMenuForManagedNode(views::MenuItemView* menu);
+  void BuildMenuForSupervisedNode(views::MenuItemView* menu);
 
   // Creates an entry in menu for each child node of |parent| starting at
   // |start_child_index|.
   void BuildMenu(const bookmarks::BookmarkNode* parent,
                  int start_child_index,
-                 views::MenuItemView* menu,
-                 int* next_menu_id);
+                 views::MenuItemView* menu);
 
-  // Returns true if |menu_id_| is outside the range of minimum and maximum menu
-  // ID's allowed.
-  bool IsOutsideMenuIdRange(int menu_id) const;
+  // Registers the necessary mappings for |menu| and |node|.
+  void AddMenuToMaps(views::MenuItemView* menu,
+                     const bookmarks::BookmarkNode* node);
 
   Browser* browser_;
   Profile* profile_;
 
   content::PageNavigator* page_navigator_;
 
   // Parent of menus.
   views::Widget* parent_;
 
   // Maps from menu id to BookmarkNode.
@@ skipping 10 matching lines @@
 
   // If non-NULL this is the |parent| passed to Init and is NOT owned by us.
   views::MenuItemView* parent_menu_item_;
 
   // Maps from node to menu.
   NodeToMenuMap node_to_menu_map_;
 
   // ID of the next menu item.
   int next_menu_id_;
 
-  // Minimum and maximum ID's to use for menu items.
-  const int min_menu_id_;
-  const int max_menu_id_;
-
   views::MenuDelegate* real_delegate_;
 
   // Is the model being changed?
   bool is_mutating_model_;
 
   // The location where this bookmark menu will be displayed (for UMA).
   BookmarkLaunchLocation location_;
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkMenuDelegate);
 };
 
 #endif  // CHROME_BROWSER_UI_VIEWS_BOOKMARKS_BOOKMARK_MENU_DELEGATE_H_