Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(898)

Unified Diff: chrome/browser/views/frame/browser_view_layout.h

Issue 3332022: Bunch of match preview tweaks: (Closed)
Patch Set: Add string16 include Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/views/frame/browser_view.cc ('k') | chrome/browser/views/frame/browser_view_layout.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/views/frame/browser_view_layout.h
diff --git a/chrome/browser/views/frame/browser_view_layout.h b/chrome/browser/views/frame/browser_view_layout.h
index 3175bdf6baa46b95e46fbaf3898fc6592d4dc709..42652efa99522f822545796008bb6083ccee6615 100644
--- a/chrome/browser/views/frame/browser_view_layout.h
+++ b/chrome/browser/views/frame/browser_view_layout.h
@@ -6,9 +6,16 @@
#define CHROME_BROWSER_VIEWS_FRAME_BROWSER_VIEW_LAYOUT_H_
#pragma once
-#include "chrome/browser/views/frame/browser_view.h"
#include "views/layout_manager.h"
+class BaseTabStrip;
+class BookmarkBarView;
+class Browser;
+class BrowserView;
+class ContentsContainer;
+class DownloadShelfView;
+class ToolbarView;
+
// The layout manager used in chrome browser.
class BrowserViewLayout : public views::LayoutManager {
public:
@@ -40,12 +47,8 @@ class BrowserViewLayout : public views::LayoutManager {
virtual gfx::Size GetPreferredSize(views::View* host);
protected:
- Browser* browser() {
- return browser_view_->browser();
- }
- const Browser* browser() const {
- return browser_view_->browser();
- }
+ Browser* browser();
+ const Browser* browser() const;
// Layout the TabStrip, returns the coordinate of the bottom of the TabStrip,
// for laying out subsequent controls.
@@ -62,6 +65,11 @@ class BrowserViewLayout : public views::LayoutManager {
// |bottom|.
void LayoutTabContents(int top, int bottom);
+ // Returns the top margin to adjust the contents_container_ by. This is used
+ // to make the bookmark bar and contents_container_ overlap so that the
+ // preview contents hides the bookmark bar.
+ int GetTopMarginForActiveContent();
+
// Layout the Download Shelf, returns the coordinate of the top of the
// control, for laying out the previous control.
int LayoutDownloadShelf(int bottom);
@@ -81,7 +89,7 @@ class BrowserViewLayout : public views::LayoutManager {
BaseTabStrip* tabstrip_;
ToolbarView* toolbar_;
views::View* contents_split_;
- views::View* contents_container_;
+ ContentsContainer* contents_container_;
views::View* infobar_container_;
DownloadShelfView* download_shelf_;
BookmarkBarView* active_bookmark_bar_;
« no previous file with comments | « chrome/browser/views/frame/browser_view.cc ('k') | chrome/browser/views/frame/browser_view_layout.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698