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

Unified Diff: chrome/browser/ui/views/toolbar_view.h

Issue 10630013: Adds LocationBarContainer that contains (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Removed debugging code Created 8 years, 6 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
Index: chrome/browser/ui/views/toolbar_view.h
diff --git a/chrome/browser/ui/views/toolbar_view.h b/chrome/browser/ui/views/toolbar_view.h
index 8bbd22c0000e6264e58d4e06c2265667e268321c..a1d878d9d6112d6c7b6738e3e22e3f5b0d09da64 100644
--- a/chrome/browser/ui/views/toolbar_view.h
+++ b/chrome/browser/ui/views/toolbar_view.h
@@ -25,6 +25,7 @@
class BrowserActionsContainer;
class Browser;
+class LocationBarContainer;
class WrenchMenu;
namespace views {
@@ -46,8 +47,9 @@ class ToolbarView : public views::AccessiblePaneView,
explicit ToolbarView(Browser* browser);
virtual ~ToolbarView();
- // Create the contents of the Browser Toolbar
- void Init();
+ // Create the contents of the Browser Toolbar. |location_bar_parent| is the
+ // view the LocationBarContainer is added to.
+ void Init(views::View* location_bar_parent);
// Updates the toolbar (and transitively the location bar) with the states of
// the specified |tab|. If |should_restore_state| is true, we're switching
@@ -80,6 +82,9 @@ class ToolbarView : public views::AccessiblePaneView,
BrowserActionsContainer* browser_actions() const { return browser_actions_; }
ReloadButton* reload_button() const { return reload_; }
LocationBarView* location_bar() const { return location_bar_; }
+ LocationBarContainer* location_bar_container() const {
+ return location_bar_container_;
+ }
views::MenuButton* app_menu() const { return app_menu_; }
// Overridden from AccessiblePaneView
@@ -184,6 +189,10 @@ class ToolbarView : public views::AccessiblePaneView,
// unacknowledged background pages in the system.
gfx::ImageSkia GetBackgroundPageBadge();
+ // Sets the bounds of the LocationBarContainer. |bounds| is in the coordinates
+ // of |this|.
+ void SetLocationBarContainerBounds(const gfx::Rect& bounds);
+
scoped_ptr<BackForwardMenuModel> back_menu_model_;
scoped_ptr<BackForwardMenuModel> forward_menu_model_;
@@ -196,6 +205,7 @@ class ToolbarView : public views::AccessiblePaneView,
ReloadButton* reload_;
views::ImageButton* home_;
LocationBarView* location_bar_;
+ LocationBarContainer* location_bar_container_;
BrowserActionsContainer* browser_actions_;
views::MenuButton* app_menu_;
Browser* browser_;

Powered by Google App Engine
This is Rietveld 408576698