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

Unified Diff: chrome/browser/ui/cocoa/extensions/browser_actions_controller.h

Issue 670463004: Make a platform-independent ToolbarActionViewController (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
diff --git a/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h b/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
index 92b16b9b2d3e18622e93815aeb3cd72a065a13a1..e6649be0b741a47540aa6e8771346f9c94d1df03 100644
--- a/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
+++ b/chrome/browser/ui/cocoa/extensions/browser_actions_controller.h
@@ -23,6 +23,10 @@ class Extension;
class ExtensionToolbarModel;
}
+namespace content {
+class WebContents;
+}
+
// Sent when the visibility of the Browser Actions changes.
extern NSString* const kBrowserActionVisibilityChangedNotification;
@@ -85,19 +89,14 @@ extern NSString* const kBrowserActionVisibilityChangedNotification;
// account the size of the grippy. Also updates the persistent width preference.
- (void)resizeContainerAndAnimate:(BOOL)animate;
-// Returns the NSView for the action button associated with an extension.
-- (NSView*)browserActionViewForExtension:(
- const extensions::Extension*)extension;
-
// Returns the saved width determined by the number of shown Browser Actions
// preference property. If no preference is found, then the width for the
// container is returned as if all buttons are shown.
- (CGFloat)savedWidth;
-// Returns where the popup arrow should point to for a given Browser Action. If
-// it is passed an extension that is not a Browser Action, then it will return
-// NSZeroPoint.
-- (NSPoint)popupPointForBrowserAction:(const extensions::Extension*)extension;
+// Returns where the popup arrow should point to for the action with the given
+// |id|. If passed an id with no corresponding button, returns NSZeroPoint.
+- (NSPoint)popupPointForId:(const std::string&)id;
// Returns whether the chevron button is currently hidden or in the process of
// being hidden (fading out). Will return NO if it is not hidden or is in the
@@ -107,6 +106,9 @@ extern NSString* const kBrowserActionVisibilityChangedNotification;
// Activates the browser action for the extension that has the given id.
- (void)activateBrowserAction:(const std::string&)extension_id;
+// Returns the currently-active web contents.
+- (content::WebContents*)currentWebContents;
+
@end // @interface BrowserActionsController
@interface BrowserActionsController(TestingAPI)

Powered by Google App Engine
This is Rietveld 408576698