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

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

Issue 246037: Integrate browser actions with the wrench menu. Browser actions (Closed)
Patch Set: alphabetize Created 11 years, 2 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
« no previous file with comments | « chrome/browser/views/browser_actions_container.cc ('k') | chrome/browser/views/toolbar_view.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "base/ref_counted.h" 10 #include "base/ref_counted.h"
(...skipping 85 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 // (nav_left true means navigation right to left, and vice versa). -1 finds 96 // (nav_left true means navigation right to left, and vice versa). -1 finds
97 // first accessible child, based on the above policy. 97 // first accessible child, based on the above policy.
98 int GetNextAccessibleViewIndex(int view_index, bool nav_left); 98 int GetNextAccessibleViewIndex(int view_index, bool nav_left);
99 99
100 void set_acc_focused_view(views::View* acc_focused_view) { 100 void set_acc_focused_view(views::View* acc_focused_view) {
101 acc_focused_view_ = acc_focused_view; 101 acc_focused_view_ = acc_focused_view;
102 } 102 }
103 103
104 // Accessors... 104 // Accessors...
105 Browser* browser() const { return browser_; } 105 Browser* browser() const { return browser_; }
106 BrowserActionsContainer* browser_actions() const { return browser_actions_; }
106 ToolbarStarToggle* star_button() const { return star_; } 107 ToolbarStarToggle* star_button() const { return star_; }
107 GoButton* go_button() const { return go_; } 108 GoButton* go_button() const { return go_; }
108 LocationBarView* location_bar() const { return location_bar_; } 109 LocationBarView* location_bar() const { return location_bar_; }
109 views::MenuButton* page_menu() const { return page_menu_; } 110 views::MenuButton* page_menu() const { return page_menu_; }
110 views::MenuButton* app_menu() const { return app_menu_; } 111 views::MenuButton* app_menu() const { return app_menu_; }
111 112
112 // Overridden from Menu::BaseControllerDelegate: 113 // Overridden from Menu::BaseControllerDelegate:
113 virtual bool GetAcceleratorInfo(int id, views::Accelerator* accel); 114 virtual bool GetAcceleratorInfo(int id, views::Accelerator* accel);
114 115
115 // Overridden from views::MenuDelegate: 116 // Overridden from views::MenuDelegate:
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 240
240 // The display mode used when laying out the toolbar. 241 // The display mode used when laying out the toolbar.
241 DisplayMode display_mode_; 242 DisplayMode display_mode_;
242 243
243 // The contents of the various menus. 244 // The contents of the various menus.
244 scoped_ptr<views::SimpleMenuModel> page_menu_contents_; 245 scoped_ptr<views::SimpleMenuModel> page_menu_contents_;
245 scoped_ptr<ZoomMenuModel> zoom_menu_contents_; 246 scoped_ptr<ZoomMenuModel> zoom_menu_contents_;
246 scoped_ptr<EncodingMenuModel> encoding_menu_contents_; 247 scoped_ptr<EncodingMenuModel> encoding_menu_contents_;
247 scoped_ptr<views::SimpleMenuModel> devtools_menu_contents_; 248 scoped_ptr<views::SimpleMenuModel> devtools_menu_contents_;
248 scoped_ptr<views::SimpleMenuModel> app_menu_contents_; 249 scoped_ptr<views::SimpleMenuModel> app_menu_contents_;
250 scoped_ptr<views::SimpleMenuModel> extension_menu_contents_;
249 251
250 // TODO(beng): build these into MenuButton. 252 // TODO(beng): build these into MenuButton.
251 scoped_ptr<views::Menu2> page_menu_menu_; 253 scoped_ptr<views::Menu2> page_menu_menu_;
252 scoped_ptr<views::Menu2> app_menu_menu_; 254 scoped_ptr<views::Menu2> app_menu_menu_;
253 }; 255 };
254 256
255 #endif // CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_ 257 #endif // CHROME_BROWSER_VIEWS_TOOLBAR_VIEW_H_
OLDNEW
« no previous file with comments | « chrome/browser/views/browser_actions_container.cc ('k') | chrome/browser/views/toolbar_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698