diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-24 19:18:48 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-24 19:18:48 +0000 |
| commit | 09b2934d0d6a35201ea68cc50eec2a4994c96c97 (patch) | |
| tree | f981f471eac7b79a8bbca4062d16310bd78f0c04 /chrome/browser/ui/cocoa/browser_window_cocoa.mm | |
| parent | 2ac71f72077ecca92e97f050d0e1574046d44be5 (diff) | |
| download | chromium_src-09b2934d0d6a35201ea68cc50eec2a4994c96c97.zip chromium_src-09b2934d0d6a35201ea68cc50eec2a4994c96c97.tar.gz chromium_src-09b2934d0d6a35201ea68cc50eec2a4994c96c97.tar.bz2 | |
Consolidates managing of bookmark bar state to browser. Browser now
pushes that state to the BrowserWindow when it changes (except during
tab switches, when it's assumed the browserwindow polls). The problem
with the current code is that as the BookmarkBar queries for state
during painting and layout, it was possible for that state without the
bookmarkbar being layed out. This resulted in the bookmark bar
painting attached even when it was given the detached state.
I'm also plumbing through one case of when the bookmark bar state
changes that wasn't being handled.
BUG=87137
TEST=see bug, but make sure you don't notice any problems with
bookmark bar visibility changes (such as toggling visibility, or
turning off and navigating back/to the new tab page).
R=pkasting@chromium.org,brettw@chromium.org,estade@chromium.org
TBR=pinkerton@chromium.org, dimich@google.com
Review URL: http://codereview.chromium.org/7237001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@90405 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ui/cocoa/browser_window_cocoa.mm')
| -rw-r--r-- | chrome/browser/ui/cocoa/browser_window_cocoa.mm | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/chrome/browser/ui/cocoa/browser_window_cocoa.mm b/chrome/browser/ui/cocoa/browser_window_cocoa.mm index fe6dd8a..05718aa 100644 --- a/chrome/browser/ui/cocoa/browser_window_cocoa.mm +++ b/chrome/browser/ui/cocoa/browser_window_cocoa.mm @@ -174,11 +174,9 @@ void BrowserWindowCocoa::UpdateTitleBar() { modes:[NSArray arrayWithObject:NSDefaultRunLoopMode]]; } -void BrowserWindowCocoa::ShelfVisibilityChanged() { - // Mac doesn't yet support showing the bookmark bar at a different size on - // the new tab page. When it does, this method should attempt to relayout the - // bookmark bar/extension shelf as their preferred height may have changed. - // http://crbug.com/43346 +void BrowserWindowCocoa::BookmarkBarStateChanged( + BookmarkBar::AnimateChangeType change_type) { + // TODO: route changes to state through this. } void BrowserWindowCocoa::UpdateDevTools() { |