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

Unified Diff: chrome/browser/cocoa/toolbar_controller.h

Issue 465130: Share the code that builds the page menu in a common model, make Mac and Win ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 11 years 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/cocoa/menu_controller_unittest.mm ('k') | chrome/browser/cocoa/toolbar_controller.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/cocoa/toolbar_controller.h
===================================================================
--- chrome/browser/cocoa/toolbar_controller.h (revision 33814)
+++ chrome/browser/cocoa/toolbar_controller.h (working copy)
@@ -25,7 +25,10 @@
class LocationBar;
class LocationBarViewMac;
@class MenuButton;
+@class MenuController;
+class PageMenuModel;
namespace ToolbarControllerInternal {
+class MenuDelegate;
class PrefObserverBridge;
}
class Profile;
@@ -52,6 +55,13 @@
scoped_nsobject<BackForwardMenuController> forwardMenuController_;
scoped_nsobject<BrowserActionsController> browserActionsController_;
+ // Lazily-instantiated model, controller, and delegate for the menu on the
+ // page button. If it's visible, these will be non-null, but they are not
+ // reaped when the button is hidden once it is initially shown.
+ scoped_ptr<PageMenuModel> pageMenuModel_;
+ scoped_nsobject<MenuController> pageMenuController_;
+ scoped_ptr<ToolbarControllerInternal::MenuDelegate> pageMenuDelegate_;
+
// Used for monitoring the optional toolbar button prefs.
scoped_ptr<ToolbarControllerInternal::PrefObserverBridge> prefObserver_;
// Used to position the omnibox bubble.
@@ -86,7 +96,6 @@
IBOutlet MenuButton* pageButton_;
IBOutlet MenuButton* wrenchButton_;
IBOutlet AutocompleteTextField* locationBar_;
- IBOutlet NSMenu* encodingMenu_;
IBOutlet NSView* browserActionContainerView_;
}
« no previous file with comments | « chrome/browser/cocoa/menu_controller_unittest.mm ('k') | chrome/browser/cocoa/toolbar_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698