diff options
| author | finnur@google.com <finnur@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-04 20:30:00 +0000 |
|---|---|---|
| committer | finnur@google.com <finnur@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-04 20:30:00 +0000 |
| commit | 550a2a688863effa5053fe426d4711d6537866bc (patch) | |
| tree | 75829dbb6da399b2ab5fbb3f8a26658e5bcf94d7 /chrome/browser/xp_frame.cc | |
| parent | ccb898ec6d8ef73ea91ced8b96a5834d2d7a9c3b (diff) | |
| download | chromium_src-550a2a688863effa5053fe426d4711d6537866bc.zip chromium_src-550a2a688863effa5053fe426d4711d6537866bc.tar.gz chromium_src-550a2a688863effa5053fe426d4711d6537866bc.tar.bz2 | |
Fixing issue 1343052: a regression where the Find box moves up when you open a new tab and close it again.
This is because during a TabClosingAt call, we end up calling ShowTabContents(NULL) which leads to ShelfVisibilityChangedImpl(NULL). Because the parameter current_tab is NULL, we end up removing the BookmarkBarView from the RootView (only to add it back a little too late), the Find window thinks there is no BookmarkBar and hence moves up a bit.
I am also enabling the disabled UI test I wrote for this, since it should now pass.
BUG=1343052
TEST=Covered by UI test.
Review URL: http://codereview.chromium.org/441
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@1727 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/xp_frame.cc')
| -rw-r--r-- | chrome/browser/xp_frame.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/xp_frame.cc b/chrome/browser/xp_frame.cc index cba5906..872acac 100644 --- a/chrome/browser/xp_frame.cc +++ b/chrome/browser/xp_frame.cc @@ -2483,7 +2483,7 @@ void XPFrame::ShelfVisibilityChangedImpl(TabContents* current_tab) { changed |= UpdateChildViewAndLayout(new_info_bar, &info_bar_view_); ChromeViews::View* new_bookmark_bar_view = NULL; - if (SupportsBookmarkBar() && current_tab) + if (SupportsBookmarkBar()) new_bookmark_bar_view = GetBookmarkBarView(); changed |= UpdateChildViewAndLayout(new_bookmark_bar_view, &active_bookmark_bar_); |