diff options
| author | glen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-12 01:40:21 +0000 |
|---|---|---|
| committer | glen@chromium.org <glen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-12-12 01:40:21 +0000 |
| commit | 6ed4b93f7cf53af95c12e0e54a167b7f99028a82 (patch) | |
| tree | c68c52c3c23d301146353d3b7128556e08956a6a /chrome/browser/views/find_bar_win.cc | |
| parent | 962c7f7d99ef92ed5339ae384a7b13d41ee38b95 (diff) | |
| download | chromium_src-6ed4b93f7cf53af95c12e0e54a167b7f99028a82.zip chromium_src-6ed4b93f7cf53af95c12e0e54a167b7f99028a82.tar.gz chromium_src-6ed4b93f7cf53af95c12e0e54a167b7f99028a82.tar.bz2 | |
Reduce the amount of space between the location bar and the bookmark bar by allowing the bookmark bar to overlap the toolbar when 'always shown'. This reduces the total height of the toolbar stack by 4 pixels and makes it look better.
Review URL: http://codereview.chromium.org/12668
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@6868 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/find_bar_win.cc')
| -rw-r--r-- | chrome/browser/views/find_bar_win.cc | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/chrome/browser/views/find_bar_win.cc b/chrome/browser/views/find_bar_win.cc index 4df9801..0510762 100644 --- a/chrome/browser/views/find_bar_win.cc +++ b/chrome/browser/views/find_bar_win.cc @@ -506,7 +506,7 @@ void FindBarWin::GetDialogBounds(gfx::Rect* bounds) { // If we're not in the New Tab page style, align ourselves with // the bookmarks bar (this works even if the bar is hidden). - if (!bookmark_bar->IsNewTabPage() || + if (!bookmark_bar->OnNewTabPage() || bookmark_bar->IsAlwaysShown()) { bookmark_bar_bounds = bookmark_bar->bounds(); } @@ -526,8 +526,10 @@ void FindBarWin::GetDialogBounds(gfx::Rect* bounds) { y_pos_offset = toolbar_bounds.bottom() - 1; // If there is a bookmark bar attached to the toolbar we should appear // attached to it instead of the toolbar. - if (!bookmark_bar_bounds.IsEmpty()) - y_pos_offset += bookmark_bar_bounds.height() - 1; + if (!bookmark_bar_bounds.IsEmpty()) { + y_pos_offset += bookmark_bar_bounds.height() - 1 - + bookmark_bar->GetToolbarOverlap(); + } } else { // There is no toolbar, so this is probably a constrained window or a Chrome // Application. This means we draw the Find window at the top of the page |