[Mac] Refactor bookmark bar controller

chrome/browser/ui/cocoa/bookmarks/bookmark_bar_controller.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_COCOA_BOOKMARKS_BOOKMARK_BAR_CONTROLLER_H_
 #define CHROME_BROWSER_UI_COCOA_BOOKMARKS_BOOKMARK_BAR_CONTROLLER_H_
 
 #import <Cocoa/Cocoa.h>
 #include <stdint.h>
 
 #include <map>
 #include <memory>
+#include <unordered_map>
 
 #import "base/mac/cocoa_protocols.h"
 #include "base/mac/scoped_nsobject.h"
 #include "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_bridge.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_constants.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_state.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_bar_toolbar_view.h"
 #import "chrome/browser/ui/cocoa/bookmarks/bookmark_button.h"
 #import "chrome/browser/ui/cocoa/has_weak_browser_pointer.h"
 #include "chrome/browser/ui/cocoa/tabs/tab_strip_model_observer_bridge.h"
@@ skipping 103 matching lines @@
 // Folder3.  Without this delay, that'll cause Sub to be closed before
 // you get there, since a "hover over" of Folder3 gets activated.
 // It's subtle but without the delay it feels broken.
 //
 // This is only really a problem with vertical menu --> vertical menu
 // movement; the bookmark bar (horizontal menu, sort of) seems fine,
 // perhaps because mouse move direction is purely vertical so there is
 // no opportunity for overlap.
 const NSTimeInterval kDragHoverCloseDelay = 0.4;
 
+enum BookmarkBarVisibleElementsMask {
+  kVisibleElementsMaskNone = 0,
+  kVisibleElementsMaskAppsButton = 1 << 0,
+  kVisibleElementsMaskManagedBookmarksButton = 1 << 1,
+  kVisibleElementsMaskSupervisedBookmarksButton = 1 << 2,
+  kVisibleElementsMaskOffTheSideButton = 1 << 3,
+  kVisibleElementsMaskOtherBookmarksButton = 1 << 4,
+  kVisibleElementsMaskNoItemTextField = 1 << 5,
+  kVisibleElementsMaskImportBookmarksButton = 1 << 6,
+};
+
+// Specifies the location and visibility of the various sub-elements
+// of the bookmark bar. Allows calculating the layout and actually
+// applying it to views to be decoupled. For example, applying
+// the layout in an RTL context transforms all horizontal offsets
+// transparently.
+struct BookmarkBarLayout {
+ public:
+  BookmarkBarLayout();
+  ~BookmarkBarLayout();
+  BookmarkBarLayout(BookmarkBarLayout&& other);
+  BookmarkBarLayout& operator=(BookmarkBarLayout&& other);
+
+  bool IsAppsButtonVisible() const {
+    return visible_elements & kVisibleElementsMaskAppsButton;
+  }
+  bool IsManagedBookmarksButtonVisible() const {
+    return visible_elements & kVisibleElementsMaskManagedBookmarksButton;
+  }
+  bool IsSupervisedBookmarksButtonVisible() const {
+    return visible_elements & kVisibleElementsMaskSupervisedBookmarksButton;
+  }
+  bool IsOffTheSideButtonVisible() const {
+    return visible_elements & kVisibleElementsMaskOffTheSideButton;
+  }
+  bool IsOtherBookmarksButtonVisible() const {
+    return visible_elements & kVisibleElementsMaskOtherBookmarksButton;
+  }
+  bool IsNoItemTextFieldVisible() const {
+    return visible_elements & kVisibleElementsMaskNoItemTextField;
+  }
+  bool IsImportBookmarksButtonVisible() const {
+    return visible_elements & kVisibleElementsMaskImportBookmarksButton;
+  }
+  size_t VisibleButtonCount() const { return button_offsets.size(); }
+
+  unsigned int visible_elements;
+  CGFloat apps_button_offset;
+  CGFloat managed_bookmarks_button_offset;
+  CGFloat supervised_bookmarks_button_offset;
+  CGFloat off_the_side_button_offset;
+  CGFloat other_bookmarks_button_offset;
+  CGFloat no_item_textfield_offset;
+  CGFloat no_item_textfield_width;
+  CGFloat import_bookmarks_button_offset;
+  CGFloat import_bookmarks_button_width;
+  CGFloat max_x;
+
+  std::unordered_map<int64_t, CGFloat> button_offsets;
+};
+
 }  // namespace bookmarks
 
 // The interface for the bookmark bar controller's delegate. Currently, the
 // delegate is the BWC and is responsible for ensuring that the toolbar is
 // displayed correctly (as specified by |-getDesiredToolbarHeightCompression|
 // and |-toolbarDividerOpacity|) at the beginning and at the end of an animation
 // (or after a state change).
 @protocol BookmarkBarControllerDelegate
 
 // Sent when the state has changed (after any animation), but before the final
@@ skipping 73 matching lines @@
   BookmarkBarFolderController* folderController_;
 
   // The event tap that allows monitoring of all events, to properly close with
   // a click outside the bounds of the window.
   id exitEventTap_;
 
   base::scoped_nsobject<BookmarkBarView>
       buttonView_;  // Contains 'no items' text fields.
   base::scoped_nsobject<BookmarkButton> offTheSideButton_;  // aka the chevron.
 
-  NSRect originalNoItemsRect_;  // Original, pre-resized field rect.
-  NSRect originalImportBookmarksRect_;  // Original, pre-resized field rect.
-
   // "Apps" button on the left side.
   base::scoped_nsobject<BookmarkButton> appsPageShortcutButton_;
 
   // "Managed bookmarks" button on the left side, next to the apps button.
   base::scoped_nsobject<BookmarkButton> managedBookmarksButton_;
 
   // "Supervised bookmarks" button on the left side, next to the apps button.
   base::scoped_nsobject<BookmarkButton> supervisedBookmarksButton_;
 
   // "Other bookmarks" button on the right side.
   base::scoped_nsobject<BookmarkButton> otherBookmarksButton_;
 
   // When doing a drag, this is folder button "hovered over" which we
   // may want to open after a short delay.  There are cases where a
   // mouse-enter can open a folder (e.g. if the menus are "active")
   // but that doesn't use this variable or need a delay so "hover" is
   // the wrong term.
   base::scoped_nsobject<BookmarkButton> hoverButton_;
 
   // We save the view width when we add bookmark buttons.  This lets
   // us avoid a rebuild until we've grown the window bigger than our
   // initial build.
   CGFloat savedFrameWidth_;
 
-  // The number of buttons we display in the bookmark bar.  This does
-  // not include the "off the side" chevron or the "Other Bookmarks"
-  // button.  We use this number to determine if we need to display
-  // the chevron, and to know what to place in the chevron's menu.
-  // Since we create everything before doing layout we can't be sure
-  // that all bookmark buttons we create will be visible.  Thus,
-  // [buttons_ count] isn't a definitive check.
-  int displayedButtonCount_;
-
   // A state flag which tracks when the bar's folder menus should be shown.
   // An initial click in any of the folder buttons turns this on and
   // one of the following will turn it off: another click in the button,
   // the window losing focus, a click somewhere other than in the bar
   // or a folder menu.
   BOOL showFolderMenus_;
 
   // If YES then state changes (for example, from hidden to shown) are animated.
   // This is turned off for unit tests.
   BOOL stateAnimationsEnabled_;
@@ skipping 145 matching lines @@
              parent:(const bookmarks::BookmarkNode*)oldParent
               index:(int)index;
 - (void)nodeFaviconLoaded:(bookmarks::BookmarkModel*)model
                      node:(const bookmarks::BookmarkNode*)node;
 - (void)nodeChildrenReordered:(bookmarks::BookmarkModel*)model
                          node:(const bookmarks::BookmarkNode*)node;
 @end
 
 // These APIs should only be used by unit tests (or used internally).
 @interface BookmarkBarController(InternalOrTestingAPI)
+- (bookmarks::BookmarkBarLayout)layoutFromCurrentState;
+- (void)applyLayout:(const bookmarks::BookmarkBarLayout&)layout
+           animated:(BOOL)animated;
 - (void)openBookmarkFolder:(id)sender;
 - (void)openOrCloseBookmarkFolderForOffTheSideButton;
 - (BookmarkBarView*)buttonView;
 - (NSMutableArray*)buttons;
-- (BOOL)offTheSideButtonIsHidden;
-- (BOOL)appsPageShortcutButtonIsHidden;
+- (BookmarkButton*)otherBookmarksButton;
+- (BookmarkButton*)managedBookmarksButton;
+- (BookmarkButton*)supervisedBookmarksButton;
 - (BookmarkButton*)otherBookmarksButton;
 - (BookmarkBarFolderController*)folderController;
 - (id)folderTarget;
-- (int)displayedButtonCount;
 - (void)openURL:(GURL)url disposition:(WindowOpenDisposition)disposition;
 - (void)clearBookmarkBar;
 - (BookmarkButtonCell*)cellForBookmarkNode:(const bookmarks::BookmarkNode*)node;
 - (BookmarkButtonCell*)cellForCustomButtonWithText:(NSString*)text
                                              image:(NSImage*)image;
-- (NSRect)frameForBookmarkButtonFromCell:(NSCell*)cell xOffset:(int*)xOffset;
 - (void)checkForBookmarkButtonGrowth:(NSButton*)button;
 - (void)frameDidChange;
 - (void)updateTheme:(const ui::ThemeProvider*)themeProvider;
 - (BookmarkButton*)buttonForDroppingOnAtPoint:(NSPoint)point;
 - (BOOL)isEventAnExitEvent:(NSEvent*)event;
-- (BOOL)shrinkOrHideView:(NSView*)view forMaxX:(CGFloat)maxViewX;
 - (void)unhighlightBookmark:(const bookmarks::BookmarkNode*)node;
 
 @end
 
 #endif  // CHROME_BROWSER_UI_COCOA_BOOKMARKS_BOOKMARK_BAR_CONTROLLER_H_