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

Side by Side Diff: chrome/browser/views/toolbar_view.h

Issue 501168: Make back forward menu model a MenuModel.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 11 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_ 5 #ifndef CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_
6 #define CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_ 6 #define CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_
7 7
8 #include <vector> 8 #include <vector>
9 9
10 #include "app/menus/simple_menu_model.h" 10 #include "app/menus/simple_menu_model.h"
11 #include "base/scoped_ptr.h" 11 #include "base/scoped_ptr.h"
12 #include "chrome/browser/app_menu_model.h" 12 #include "chrome/browser/app_menu_model.h"
13 #include "chrome/browser/back_forward_menu_model_views.h" 13 #include "chrome/browser/back_forward_menu_model.h"
14 #include "chrome/browser/bubble_positioner.h" 14 #include "chrome/browser/bubble_positioner.h"
15 #include "chrome/browser/command_updater.h" 15 #include "chrome/browser/command_updater.h"
16 #include "chrome/browser/page_menu_model.h" 16 #include "chrome/browser/page_menu_model.h"
17 #include "chrome/browser/views/accessible_toolbar_view.h" 17 #include "chrome/browser/views/accessible_toolbar_view.h"
18 #include "chrome/browser/views/go_button.h" 18 #include "chrome/browser/views/go_button.h"
19 #include "chrome/browser/views/location_bar_view.h" 19 #include "chrome/browser/views/location_bar_view.h"
20 #include "chrome/common/pref_member.h" 20 #include "chrome/common/pref_member.h"
21 #include "views/controls/button/menu_button.h" 21 #include "views/controls/button/menu_button.h"
22 #include "views/controls/menu/menu.h" 22 #include "views/controls/menu/menu.h"
23 #include "views/controls/menu/view_menu_delegate.h" 23 #include "views/controls/menu/view_menu_delegate.h"
(...skipping 117 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 // Types of display mode this toolbar can have. 141 // Types of display mode this toolbar can have.
142 enum DisplayMode { 142 enum DisplayMode {
143 DISPLAYMODE_NORMAL, // Normal toolbar with buttons, etc. 143 DISPLAYMODE_NORMAL, // Normal toolbar with buttons, etc.
144 DISPLAYMODE_LOCATION // Slimline toolbar showing only compact location 144 DISPLAYMODE_LOCATION // Slimline toolbar showing only compact location
145 // bar, used for popups. 145 // bar, used for popups.
146 }; 146 };
147 bool IsDisplayModeNormal() const { 147 bool IsDisplayModeNormal() const {
148 return display_mode_ == DISPLAYMODE_NORMAL; 148 return display_mode_ == DISPLAYMODE_NORMAL;
149 } 149 }
150 150
151 scoped_ptr<BackForwardMenuModelViews> back_menu_model_; 151 scoped_ptr<BackForwardMenuModel> back_menu_model_;
152 scoped_ptr<BackForwardMenuModelViews> forward_menu_model_; 152 scoped_ptr<BackForwardMenuModel> forward_menu_model_;
153 153
154 // The model that contains the security level, text, icon to display... 154 // The model that contains the security level, text, icon to display...
155 ToolbarModel* model_; 155 ToolbarModel* model_;
156 156
157 // Controls 157 // Controls
158 views::ImageButton* back_; 158 views::ImageButton* back_;
159 views::ImageButton* forward_; 159 views::ImageButton* forward_;
160 views::ImageButton* reload_; 160 views::ImageButton* reload_;
161 views::ImageButton* home_; 161 views::ImageButton* home_;
162 ToolbarStarToggle* star_; 162 ToolbarStarToggle* star_;
(...skipping 19 matching lines...) Expand all
182 // The contents of the various menus. 182 // The contents of the various menus.
183 scoped_ptr<PageMenuModel> page_menu_model_; 183 scoped_ptr<PageMenuModel> page_menu_model_;
184 scoped_ptr<AppMenuModel> app_menu_model_; 184 scoped_ptr<AppMenuModel> app_menu_model_;
185 185
186 // TODO(beng): build these into MenuButton. 186 // TODO(beng): build these into MenuButton.
187 scoped_ptr<views::Menu2> page_menu_menu_; 187 scoped_ptr<views::Menu2> page_menu_menu_;
188 scoped_ptr<views::Menu2> app_menu_menu_; 188 scoped_ptr<views::Menu2> app_menu_menu_;
189 }; 189 };
190 190
191 #endif // CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_ 191 #endif // CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698