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

Side by Side Diff: chrome/browser/cocoa/toolbar_controller.h

Issue 3163023: Clean up the WrenchMenuModel so that it uses SimpleMenu::Delegate. (Closed) Base URL: http://src.chromium.org/git/chromium.git
Patch Set: Fix stray rb Created 10 years, 4 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
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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_COCOA_TOOLBAR_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_
6 #define CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_ 6 #define CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_
7 #pragma once 7 #pragma once
8 8
9 #import <Cocoa/Cocoa.h> 9 #import <Cocoa/Cocoa.h>
10 10
(...skipping 10 matching lines...) Expand all
21 @class BrowserActionsContainerView; 21 @class BrowserActionsContainerView;
22 @class BackForwardMenuController; 22 @class BackForwardMenuController;
23 class Browser; 23 class Browser;
24 @class BrowserActionsController; 24 @class BrowserActionsController;
25 class CommandUpdater; 25 class CommandUpdater;
26 @class DelayedMenuButton; 26 @class DelayedMenuButton;
27 class LocationBar; 27 class LocationBar;
28 class LocationBarViewMac; 28 class LocationBarViewMac;
29 @class MenuButton; 29 @class MenuButton;
30 namespace ToolbarControllerInternal { 30 namespace ToolbarControllerInternal {
31 class MenuDelegate; 31 class WrenchAcceleratorDelegate;
32 class NotificationBridge; 32 class NotificationBridge;
33 } // namespace ToolbarControllerInternal 33 } // namespace ToolbarControllerInternal
34 class Profile; 34 class Profile;
35 @class ReloadButton; 35 @class ReloadButton;
36 class TabContents; 36 class TabContents;
37 class ToolbarModel; 37 class ToolbarModel;
38 @class WrenchMenuController; 38 @class WrenchMenuController;
39 class WrenchMenuModel; 39 class WrenchMenuModel;
40 40
41 // A controller for the toolbar in the browser window. Manages 41 // A controller for the toolbar in the browser window. Manages
(...skipping 25 matching lines...) Expand all
67 scoped_ptr<LocationBarViewMac> locationBarView_; 67 scoped_ptr<LocationBarViewMac> locationBarView_;
68 scoped_nsobject<AutocompleteTextFieldEditor> autocompleteTextFieldEditor_; 68 scoped_nsobject<AutocompleteTextFieldEditor> autocompleteTextFieldEditor_;
69 id<ViewResizer> resizeDelegate_; // weak 69 id<ViewResizer> resizeDelegate_; // weak
70 scoped_nsobject<BackForwardMenuController> backMenuController_; 70 scoped_nsobject<BackForwardMenuController> backMenuController_;
71 scoped_nsobject<BackForwardMenuController> forwardMenuController_; 71 scoped_nsobject<BackForwardMenuController> forwardMenuController_;
72 scoped_nsobject<BrowserActionsController> browserActionsController_; 72 scoped_nsobject<BrowserActionsController> browserActionsController_;
73 73
74 // Lazily-instantiated model and delegate for the menu on the 74 // Lazily-instantiated model and delegate for the menu on the
75 // wrench button. Once visible, it will be non-null, but will not 75 // wrench button. Once visible, it will be non-null, but will not
76 // reaped when the menu is hidden once it is initially shown. 76 // reaped when the menu is hidden once it is initially shown.
77 scoped_ptr<ToolbarControllerInternal::MenuDelegate> menuDelegate_; 77 scoped_ptr<ToolbarControllerInternal::WrenchAcceleratorDelegate>
78 acceleratorDelegate_;
78 scoped_ptr<WrenchMenuModel> wrenchMenuModel_; 79 scoped_ptr<WrenchMenuModel> wrenchMenuModel_;
79 80
80 // Used for monitoring the optional toolbar button prefs. 81 // Used for monitoring the optional toolbar button prefs.
81 scoped_ptr<ToolbarControllerInternal::NotificationBridge> notificationBridge_; 82 scoped_ptr<ToolbarControllerInternal::NotificationBridge> notificationBridge_;
82 BooleanPrefMember showHomeButton_; 83 BooleanPrefMember showHomeButton_;
83 BooleanPrefMember showPageOptionButtons_; 84 BooleanPrefMember showPageOptionButtons_;
84 BOOL hasToolbar_; // If NO, we may have only the location bar. 85 BOOL hasToolbar_; // If NO, we may have only the location bar.
85 BOOL hasLocationBar_; // If |hasToolbar_| is YES, this must also be YES. 86 BOOL hasLocationBar_; // If |hasToolbar_| is YES, this must also be YES.
86 BOOL locationBarAtMinSize_; // If the location bar is at the minimum size. 87 BOOL locationBarAtMinSize_; // If the location bar is at the minimum size.
87 88
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
179 // Returns an array of views in the order of the outlets above. 180 // Returns an array of views in the order of the outlets above.
180 - (NSArray*)toolbarViews; 181 - (NSArray*)toolbarViews;
181 - (void)showOptionalHomeButton; 182 - (void)showOptionalHomeButton;
182 - (void)installWrenchMenu; 183 - (void)installWrenchMenu;
183 - (WrenchMenuController*)wrenchMenuController; 184 - (WrenchMenuController*)wrenchMenuController;
184 // Return a hover button for the current event. 185 // Return a hover button for the current event.
185 - (NSButton*)hoverButtonForEvent:(NSEvent*)theEvent; 186 - (NSButton*)hoverButtonForEvent:(NSEvent*)theEvent;
186 @end 187 @end
187 188
188 #endif // CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_ 189 #endif // CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698