[Mac] Implement LocationBarViewMac::UpdateLocationBarVisibility()

chrome/browser/ui/cocoa/browser_window_layout.h

Index: chrome/browser/ui/cocoa/browser_window_layout.h
diff --git a/chrome/browser/ui/cocoa/browser_window_layout.h b/chrome/browser/ui/cocoa/browser_window_layout.h
index 82cf1f9cfe6cf6148d50e30600a44477aa9e3379..13f22bfb95159cc9eacd4f432b93c560f6d3cfc1 100644
--- a/chrome/browser/ui/cocoa/browser_window_layout.h
+++ b/chrome/browser/ui/cocoa/browser_window_layout.h
@@ -52,6 +52,9 @@ struct LayoutParameters {
   // superview. Only needs to be set when shouldShowAvatar is YES.
   CGFloat avatarLineWidth;
 

[tapted, 2015/10/12 23:56:43]

nit: remove newline

+  // Hosted apps hide and show the toolbar (location bar) independently of
+  // whether it is available as a browser feature.
+  BOOL toolbarHidden;

[tapted, 2015/10/12 23:56:43]

nit: newline after (otherwise it looks like the comment applies to all 3 params)

[tapted, 2015/10/13 00:00:19]

Also, maybe this can be obsoleted by setting toolbarHeight below to 0?

(perhaps it should be called `targetToolbarHeight` in that case... in which case
it could perhaps also make LocationBarViewMac::visible_ obsolete? -- i.e. it
would be nicer to derive LocationBarViewMac::visible_ rather than cache it)

[dominickn, 2015/10/15 06:22:28]

Changes discarded.

   BOOL hasToolbar;
   BOOL hasLocationBar;
   CGFloat toolbarHeight;
@@ -159,6 +162,7 @@ struct LayoutOutput {
 - (void)setAvatarSize:(NSSize)avatarSize;
 - (void)setAvatarLineWidth:(CGFloat)avatarLineWidth;
 
+- (void)setToolbarHidden:(BOOL)toolbarHidden;
 - (void)setHasToolbar:(BOOL)hasToolbar;
 - (void)setHasLocationBar:(BOOL)hasLocationBar;
 - (void)setToolbarHeight:(CGFloat)toolbarHeight;