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

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

Issue 3163035: Revert "Clean up the WrenchMenuModel so that it uses SimpleMenu::Delegate." (Closed) Base URL: http://src.chromium.org/git/chromium.git
Patch Set: 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
« no previous file with comments | « app/menus/accelerator.h ('k') | chrome/browser/cocoa/toolbar_controller.mm » ('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) 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 WrenchAcceleratorDelegate; 31 class MenuDelegate;
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::WrenchAcceleratorDelegate> 77 scoped_ptr<ToolbarControllerInternal::MenuDelegate> menuDelegate_;
78 acceleratorDelegate_;
79 scoped_ptr<WrenchMenuModel> wrenchMenuModel_; 78 scoped_ptr<WrenchMenuModel> wrenchMenuModel_;
80 79
81 // Used for monitoring the optional toolbar button prefs. 80 // Used for monitoring the optional toolbar button prefs.
82 scoped_ptr<ToolbarControllerInternal::NotificationBridge> notificationBridge_; 81 scoped_ptr<ToolbarControllerInternal::NotificationBridge> notificationBridge_;
83 BooleanPrefMember showHomeButton_; 82 BooleanPrefMember showHomeButton_;
84 BooleanPrefMember showPageOptionButtons_; 83 BooleanPrefMember showPageOptionButtons_;
85 BOOL hasToolbar_; // If NO, we may have only the location bar. 84 BOOL hasToolbar_; // If NO, we may have only the location bar.
86 BOOL hasLocationBar_; // If |hasToolbar_| is YES, this must also be YES. 85 BOOL hasLocationBar_; // If |hasToolbar_| is YES, this must also be YES.
87 BOOL locationBarAtMinSize_; // If the location bar is at the minimum size. 86 BOOL locationBarAtMinSize_; // If the location bar is at the minimum size.
88 87
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 // Returns an array of views in the order of the outlets above. 179 // Returns an array of views in the order of the outlets above.
181 - (NSArray*)toolbarViews; 180 - (NSArray*)toolbarViews;
182 - (void)showOptionalHomeButton; 181 - (void)showOptionalHomeButton;
183 - (void)installWrenchMenu; 182 - (void)installWrenchMenu;
184 - (WrenchMenuController*)wrenchMenuController; 183 - (WrenchMenuController*)wrenchMenuController;
185 // Return a hover button for the current event. 184 // Return a hover button for the current event.
186 - (NSButton*)hoverButtonForEvent:(NSEvent*)theEvent; 185 - (NSButton*)hoverButtonForEvent:(NSEvent*)theEvent;
187 @end 186 @end
188 187
189 #endif // CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_ 188 #endif // CHROME_BROWSER_COCOA_TOOLBAR_CONTROLLER_H_
OLDNEW
« no previous file with comments | « app/menus/accelerator.h ('k') | chrome/browser/cocoa/toolbar_controller.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698