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

Side by Side Diff: chrome/browser/ui/cocoa/extensions/browser_actions_controller.h

Issue 1221173003: [Mac] Inform reference counted objects that hold a weak Browser* when the Browser is being destroye… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove dealloc override. Created 5 years, 5 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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_UI_COCOA_EXTENSIONS_BROWSER_ACTIONS_CONTROLLER_H_ 5 #ifndef CHROME_BROWSER_UI_COCOA_EXTENSIONS_BROWSER_ACTIONS_CONTROLLER_H_
6 #define CHROME_BROWSER_UI_COCOA_EXTENSIONS_BROWSER_ACTIONS_CONTROLLER_H_ 6 #define CHROME_BROWSER_UI_COCOA_EXTENSIONS_BROWSER_ACTIONS_CONTROLLER_H_
7 7
8 #import <Cocoa/Cocoa.h> 8 #import <Cocoa/Cocoa.h>
9 9
10 #import "base/mac/scoped_nsobject.h" 10 #import "base/mac/scoped_nsobject.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 77
78 // Initializes the controller given the current browser and container view that 78 // Initializes the controller given the current browser and container view that
79 // will hold the browser action buttons. If |mainController| is nil, the created 79 // will hold the browser action buttons. If |mainController| is nil, the created
80 // BrowserActionsController will be the main controller; otherwise (if this is 80 // BrowserActionsController will be the main controller; otherwise (if this is
81 // for the overflow menu), |mainController| should be controller of the main bar 81 // for the overflow menu), |mainController| should be controller of the main bar
82 // for the |browser|. 82 // for the |browser|.
83 - (id)initWithBrowser:(Browser*)browser 83 - (id)initWithBrowser:(Browser*)browser
84 containerView:(BrowserActionsContainerView*)container 84 containerView:(BrowserActionsContainerView*)container
85 mainController:(BrowserActionsController*)mainController; 85 mainController:(BrowserActionsController*)mainController;
86 86
87 // Called by BrowserWindowController when the Browser is being destroyed.
88 - (void)browserWillBeDestroyed;
89
87 // Update the display of all buttons. 90 // Update the display of all buttons.
88 - (void)update; 91 - (void)update;
89 92
90 // Returns the current number of browser action buttons within the container, 93 // Returns the current number of browser action buttons within the container,
91 // whether or not they are displayed. 94 // whether or not they are displayed.
92 - (NSUInteger)buttonCount; 95 - (NSUInteger)buttonCount;
93 96
94 // Returns the current number of browser action buttons displayed in the 97 // Returns the current number of browser action buttons displayed in the
95 // container. 98 // container.
96 - (NSUInteger)visibleButtonCount; 99 - (NSUInteger)visibleButtonCount;
(...skipping 23 matching lines...) Expand all
120 // Sets whether or not the overflow container is focused in the wrench menu. 123 // Sets whether or not the overflow container is focused in the wrench menu.
121 - (void)setFocusedInOverflow:(BOOL)focused; 124 - (void)setFocusedInOverflow:(BOOL)focused;
122 125
123 @end // @interface BrowserActionsController 126 @end // @interface BrowserActionsController
124 127
125 @interface BrowserActionsController(TestingAPI) 128 @interface BrowserActionsController(TestingAPI)
126 - (BrowserActionButton*)buttonWithIndex:(NSUInteger)index; 129 - (BrowserActionButton*)buttonWithIndex:(NSUInteger)index;
127 @end 130 @end
128 131
129 #endif // CHROME_BROWSER_UI_COCOA_EXTENSIONS_BROWSER_ACTIONS_CONTROLLER_H_ 132 #endif // CHROME_BROWSER_UI_COCOA_EXTENSIONS_BROWSER_ACTIONS_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698