Rewrites the Mac view resizing logic to have the BrowserWindowController...

chrome/browser/cocoa/bookmark_bar_controller.mm

 // Copyright (c) 2009 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 "base/mac_util.h"
 #include "base/sys_string_conversions.h"
 #include "chrome/browser/bookmarks/bookmark_editor.h"
 #include "chrome/browser/bookmarks/bookmark_model.h"
 #include "chrome/browser/browser.h"
 #include "chrome/browser/browser_list.h"
 #import "chrome/browser/cocoa/bookmark_bar_bridge.h"
 #import "chrome/browser/cocoa/bookmark_bar_controller.h"
 #import "chrome/browser/cocoa/bookmark_button_cell.h"
 #import "chrome/browser/cocoa/bookmark_editor_controller.h"
 #import "chrome/browser/cocoa/bookmark_name_folder_controller.h"
 #import "chrome/browser/cocoa/bookmark_menu_cocoa_controller.h"
+#import "chrome/browser/cocoa/view_resizer.h"
 #include "chrome/browser/cocoa/nsimage_cache.h"
 #include "chrome/browser/profile.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/common/pref_service.h"
 #include "skia/ext/skia_utils_mac.h"
 
 @interface BookmarkBarController(Private)
 - (void)applyContentAreaOffset:(BOOL)apply immediately:(BOOL)immediately;
 - (void)showBookmarkBar:(BOOL)enable immediately:(BOOL)immediately;
 - (void)addNode:(const BookmarkNode*)child toMenu:(NSMenu*)menu;
 - (void)addFolderNode:(const BookmarkNode*)node toMenu:(NSMenu*)menu;
 - (void)tagEmptyMenu:(NSMenu*)menu;
 - (void)clearMenuTagMap;
 @end
 
 namespace {
 
-// TODO(jrg): this is the right proportional height but overlaps the
-// "blue outline" of the omnibox.  Fix.
-
 // Our height, when opened.
 const int kBookmarkBarHeight = 30;
-// How much to adjust our parent view.
-const int kBookmarkBarSuperviewHeightAdjustment = 25;
-// How much to adjust the web frame.
-const int kBookmarkBarWebframeHeightAdjustment = 25;
 
 // Magic numbers from Cole
 const CGFloat kDefaultBookmarkWidth = 150.0;
 const CGFloat kBookmarkVerticalPadding = 2.0;
 const CGFloat kBookmarkHorizontalPadding = 1.0;
 };
 
 @implementation BookmarkBarController
 
 - (id)initWithProfile:(Profile*)profile
-           parentView:(NSView*)parentView
-       webContentView:(NSView*)webContentView
-         infoBarsView:(NSView*)infoBarsView
-             delegate:(id<BookmarkURLOpener>)delegate {
+         initialWidth:(float)initialWidth
+       resizeDelegate:(id<ViewResizer>)resizeDelegate
+          urlDelegate:(id<BookmarkURLOpener>)urlDelegate {
   if ((self = [super initWithNibName:@"BookmarkBar"
                               bundle:mac_util::MainAppBundle()])) {
     profile_ = profile;
+    initialWidth_ = initialWidth;
     bookmarkModel_ = profile->GetBookmarkModel();
-    parentView_ = parentView;
-    webContentView_ = webContentView;
-    infoBarsView_ = infoBarsView;
     buttons_.reset([[NSMutableArray alloc] init]);
-    delegate_ = delegate;
+    resizeDelegate_ = resizeDelegate;
+    urlDelegate_ = urlDelegate;
   }
   return self;
 }
 
 - (void)dealloc {
   [[NSNotificationCenter defaultCenter] removeObserver:self];
   [super dealloc];
 }
 
 - (void)awakeFromNib {
   // We default to NOT open, which means height=0.
   DCHECK([[self view] isHidden]);  // Hidden so it's OK to change.
-  NSRect frame = [[self view] frame];
-  frame.size.height = 0;
-  frame.size.width = [parentView_ frame].size.width;
-  [[self view] setFrame:frame];
 
-  // Make sure the nodes stay bottom-aligned.
-  [[self view] setAutoresizingMask:(NSViewWidthSizable |
-                                         NSViewMinYMargin)];
+  // Set our initial height to zero, since that is what the superview
+  // expects.  We will resize ourselves open later if needed.
+  [[self view] setFrame:NSMakeRect(0, 0, initialWidth_, 0)];
+
   // Be sure to enable the bar before trying to show it...
   barIsEnabled_ = YES;
   if (profile_->GetPrefs()->GetBoolean(prefs::kShowBookmarkBar))
     [self showBookmarkBar:YES immediately:YES];
 
   // Don't pass ourself along (as 'self') until our init is completely
   // done.  Thus, this call is (almost) last.
   bridge_.reset(new BookmarkBarBridge(self, bookmarkModel_));
 
   // When resized we may need to add new buttons, or remove them (if
@@ skipping 32 matching lines @@
 }
 
 // Show or hide the bar based on the value of |show|. Handles
 // animating the resize of the content view.  if |immediately| is YES,
 // make changes immediately instead of using an animator.  If the bar
 // is disabled, do absolutely nothing.  The routine which enables the
 // bar will show it if relevant using other mechanisms (the pref) to
 // determine desired state.
 - (void)showBookmarkBar:(BOOL)show immediately:(BOOL)immediately {
   if (barIsEnabled_ && (barShouldBeShown_ != show)) {
-    contentViewHasOffset_ = show;
-    [[self view] setHidden:show ? NO : YES];
+    if ([self view]) {
+      [[self view] setHidden:show ? NO : YES];
+      [resizeDelegate_ resizeView:[self view]
+                       newHeight:(show ? kBookmarkBarHeight : 0)];
+    }
     barShouldBeShown_ = show;
     if (show) {
       [self loaded:bookmarkModel_];
     }
-    [self applyContentAreaOffset:show immediately:immediately];
   }
 }
 
-// Apply a contents box offset to make (or remove) room for the
-// bookmark bar.  If apply==YES, always make room (the contentView_ is
-// "full size").  If apply==NO we are trying to undo an offset.  If no
-// offset there is nothing to undo.
-//
-// TODO(jrg): it is awkward we change the sizes of views for our
-// parent and siblings; ideally they change their own sizes.
-//
-// TODO(jrg): unlike windows, we process events while an animator is
-// running.  Thus, if you resize the window while the bookmark bar is
-// animating, you'll mess things up.  Fix.
-- (void)applyContentAreaOffset:(BOOL)apply immediately:(BOOL)immediately {
-  if ([self view] == nil) {
-    // We're too early, but awakeFromNib will call me again.
-    return;
-  }
-  if (!contentViewHasOffset_ && apply) {
-    // There is no offset to unconditionally apply.
-    return;
-  }
-
-  // None of these locals are members of the Hall of Justice.
-  NSRect superframe = [parentView_ frame];
-  NSRect frame = [[self view] frame];
-  NSRect webframe = [webContentView_ frame];
-  NSRect infoframe = [infoBarsView_ frame];
-  if (apply) {
-    superframe.size.height += kBookmarkBarSuperviewHeightAdjustment;
-    // TODO(jrg): y=0 if we add the bookmark bar before the parent
-    // view (toolbar) is placed in the view hierarchy.  A different
-    // CL, where the bookmark bar is extracted from the toolbar nib,
-    // may fix this awkwardness.
-    if (superframe.origin.y > 0) {
-      superframe.origin.y -= kBookmarkBarSuperviewHeightAdjustment;
-      webframe.size.height -= kBookmarkBarWebframeHeightAdjustment;
-    }
-    frame.size.height += kBookmarkBarHeight;
-    infoframe.origin.y -= kBookmarkBarWebframeHeightAdjustment;
-  } else {
-    superframe.size.height -= kBookmarkBarSuperviewHeightAdjustment;
-    superframe.origin.y += kBookmarkBarSuperviewHeightAdjustment;
-    frame.size.height -= kBookmarkBarHeight;
-    webframe.size.height += kBookmarkBarWebframeHeightAdjustment;
-    infoframe.origin.y += kBookmarkBarWebframeHeightAdjustment;
-  }
-
-  // TODO(jrg): Animators can be a little fussy.  Setting these three
-  // off can sometimes causes races where the finish isn't as
-  // expected.  Fix, or clean out the animators as an option.
-  // Odd racing is FAR worse than a lack of an animator.
-  if (1 /* immediately */) {
-    [parentView_ setFrame:superframe];
-    [webContentView_ setFrame:webframe];
-    [infoBarsView_ setFrame:infoframe];
-    [[self view] setFrame:frame];
-  } else {
-    [[parentView_ animator] setFrame:superframe];
-    [[webContentView_ animator] setFrame:webframe];
-    [[infoBarsView_ animator] setFrame:infoframe];
-    [[[self view] animator] setFrame:frame];
-  }
-
-  [[self view] setNeedsDisplay:YES];
-  [parentView_ setNeedsDisplay:YES];
-  [webContentView_ setNeedsDisplay:YES];
-  [infoBarsView_ setNeedsDisplay:YES];
-}
-
 - (BOOL)isBookmarkBarVisible {
   return barShouldBeShown_;
 }
 
 // We don't change a preference; we only change visibility.
 // Preference changing (global state) is handled in
 // BrowserWindowCocoa::ToggleBookmarkBar().
 - (void)toggleBookmarkBar {
   [self showBookmarkBar:!barShouldBeShown_ immediately:YES];
 }
@@ skipping 26 matching lines @@
 - (BookmarkNode*)nodeFromButton:(id)button {
   NSCell* cell = [button cell];
   BookmarkNode* node = static_cast<BookmarkNode*>(
       [[cell representedObject] pointerValue]);
   DCHECK(node);
   return node;
 }
 
 - (IBAction)openBookmark:(id)sender {
   BookmarkNode* node = [self nodeFromButton:sender];
-  [delegate_ openBookmarkURL:node->GetURL() disposition:CURRENT_TAB];
+  [urlDelegate_ openBookmarkURL:node->GetURL() disposition:CURRENT_TAB];
 }
 
 // Given a NSMenuItem tag, return the appropriate bookmark node id.
 - (int64)nodeIdFromMenuTag:(int32)tag {
   return menuTagMap_[tag];
 }
 
 // Create and return a new tag for the given node id.
 - (int32)menuTagFromNodeId:(int64)menuid {
   int tag = seedId_++;
@@ skipping 102 matching lines @@
                  withEvent:[NSApp currentEvent]
                    forView:sender];
 }
 
 // As a convention we set the menu's delegate to be the button's cell
 // so we can easily obtain bookmark info.  Convention applied in
 // -[BookmarkButtonCell menu].
 
 - (IBAction)openBookmarkInNewForegroundTab:(id)sender {
   BookmarkNode* node = [self nodeFromMenuItem:sender];
-  [delegate_ openBookmarkURL:node->GetURL() disposition:NEW_FOREGROUND_TAB];
+  [urlDelegate_ openBookmarkURL:node->GetURL() disposition:NEW_FOREGROUND_TAB];
 }
 
 - (IBAction)openBookmarkInNewWindow:(id)sender {
   BookmarkNode* node = [self nodeFromMenuItem:sender];
-  [delegate_ openBookmarkURL:node->GetURL() disposition:NEW_WINDOW];
+  [urlDelegate_ openBookmarkURL:node->GetURL() disposition:NEW_WINDOW];
 }
 
 - (IBAction)openBookmarkInIncognitoWindow:(id)sender {
   BookmarkNode* node = [self nodeFromMenuItem:sender];
-  [delegate_ openBookmarkURL:node->GetURL() disposition:OFF_THE_RECORD];
+  [urlDelegate_ openBookmarkURL:node->GetURL() disposition:OFF_THE_RECORD];
 }
 
 - (IBAction)editBookmark:(id)sender {
   BookmarkNode* node = [self nodeFromMenuItem:sender];
 
   // TODO(jrg): on windows, folder "buttons" use the bar's context
   // menu (but with extra items enabled, like Rename).  For now we do
   // a cheat and redirect so we have the functionality available.
   if (node->is_folder()) {
     [self addOrRenameFolder:sender];
@@ skipping 18 matching lines @@
 - (IBAction)deleteBookmark:(id)sender {
   BookmarkNode* node = [self nodeFromMenuItem:sender];
   bookmarkModel_->Remove(node->GetParent(),
                          node->GetParent()->IndexOfChild(node));
 }
 
 - (void)openBookmarkNodesRecursive:(BookmarkNode*)node {
   for (int i = 0; i < node->GetChildCount(); i++) {
     BookmarkNode* child = node->GetChild(i);
     if (child->is_url())
-      [delegate_ openBookmarkURL:child->GetURL()
+      [urlDelegate_ openBookmarkURL:child->GetURL()
                      disposition:NEW_BACKGROUND_TAB];
     else
       [self openBookmarkNodesRecursive:child];
   }
 }
 
 - (IBAction)openAllBookmarks:(id)sender {
   // TODO(jrg):
   // Is there an easier way to get a non-const root node for the bookmark bar?
   // I can't iterate over them unless it's non-const.
@@ skipping 120 matching lines @@
         each_frame.origin.x += delta;
         [each_button setFrame:each_frame];
       }
     }
   }
 }
 
 - (IBAction)openBookmarkMenuItem:(id)sender {
   int64 tag = [self nodeIdFromMenuTag:[sender tag]];
   const BookmarkNode* node = bookmarkModel_->GetNodeByID(tag);
-  [delegate_ openBookmarkURL:node->GetURL() disposition:CURRENT_TAB];
+  [urlDelegate_ openBookmarkURL:node->GetURL() disposition:CURRENT_TAB];
 }
 
 // Add all items from the given model to our bookmark bar.
 // TODO(jrg): lots of things!
 //  - bookmark folders (e.g. menu from the button)
 //  - button and menu on the right for when bookmarks don't all fit on the
 //    screen
 //  - ...
 //
 // TODO(jrg): write a "build bar" so there is a nice spot for things
@@ skipping 107 matching lines @@
     }
   }
 }
 
 // TODO(jrg): for now this is brute force.
 - (void)nodeChildrenReordered:(BookmarkModel*)model
                          node:(const BookmarkNode*)node {
   [self loaded:model];
 }
 
-- (void)setDelegate:(id<BookmarkURLOpener>)delegate {
-  delegate_ = delegate;
+- (void)setUrlDelegate:(id<BookmarkURLOpener>)urlDelegate {
+  urlDelegate_ = urlDelegate;
 }
 
 - (NSArray*)buttons {
   return buttons_.get();
 }
 
 @end