[Mac] Implement LocationBarViewMac::UpdateLocationBarVisibility()

chrome/browser/ui/cocoa/browser_window_controller_private.mm

Index: chrome/browser/ui/cocoa/browser_window_controller_private.mm
diff --git a/chrome/browser/ui/cocoa/browser_window_controller_private.mm b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
index d680763b610cd1935eb5c274027738e6f4afa6b5..3f6070368b6be64f6aee135f0cb2742d0d11b40b 100644
--- a/chrome/browser/ui/cocoa/browser_window_controller_private.mm
+++ b/chrome/browser/ui/cocoa/browser_window_controller_private.mm
@@ -437,6 +437,20 @@ willPositionSheet:(NSWindow*)sheet
                                delay:YES];
 }
 
+- (void)showToolbar:(NSNotification*)notification {
+  if (!shouldShowToolbar_) {
+    shouldShowToolbar_ = YES;
+    [self layoutSubviews];
+  }
+}
+
+- (void)hideToolbar:(NSNotification*)notification {
+  if (shouldShowToolbar_) {
+    shouldShowToolbar_ = NO;
+    [self layoutSubviews];
+  }
+}
+
 - (void)configurePresentationModeController {
   BOOL fullscreen_for_tab = browser_->exclusive_access_manager()
                                 ->fullscreen_controller()
@@ -901,8 +915,13 @@ willPositionSheet:(NSWindow*)sheet
     [layout setAvatarLineWidth:[[avatar superview] cr_lineWidth]];
   }
 
-  [layout setHasToolbar:[self hasToolbar]];
-  [layout setToolbarHeight:NSHeight([[toolbarController_ view] bounds])];
+  if (shouldShowToolbar_) {

[tapted, 2015/09/08 05:55:05]

I think you can read `shouldShowToolbar` by just asking the location bar whether
it's hidden (then would just need to trigger a call to layoutSubviews). But all
that should be irrelevant if we can just collapse the height of the ToolbarView
using AnimatableView.

[dominickn, 2015/10/09 06:37:02]

Done.

+    [layout setHasToolbar:[self hasToolbar]];
+    [layout setToolbarHeight:NSHeight([[toolbarController_ view] bounds])];
+  } else {
+    [layout setHasToolbar:NO];
+    [layout setToolbarHeight:0];
+  }
 
   [layout setHasLocationBar:[self hasLocationBar]];