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

Unified Diff: chrome/browser/extensions/extension_toolbar_model.h

Issue 496863003: Consolidate ExtensionAction execution code (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_toolbar_model.h
diff --git a/chrome/browser/extensions/extension_toolbar_model.h b/chrome/browser/extensions/extension_toolbar_model.h
index bf42f7ce332a3f66eed7f6d25321d17d0e1f0c52..c0d35d6aa9a35c02be70fbf122a7e4bd907c12a7 100644
--- a/chrome/browser/extensions/extension_toolbar_model.h
+++ b/chrome/browser/extensions/extension_toolbar_model.h
@@ -89,18 +89,6 @@ class ExtensionToolbarModel : public content::NotificationObserver,
// Moves the given |extension|'s icon to the given |index|.
void MoveExtensionIcon(const Extension* extension, int index);
- // Executes the browser action for an extension and returns the action that
- // the UI should perform in response.
- // |popup_url_out| will be set if the extension should show a popup, with
- // the URL that should be shown, if non-NULL. |should_grant| controls whether
- // the extension should be granted page tab permissions, which is what happens
- // when the user clicks the browser action, but not, for example, when the
- // showPopup API is called.
- ExtensionAction::ShowAction ExecuteBrowserAction(const Extension* extension,
- Browser* browser,
- GURL* popup_url_out,
- bool should_grant);
-
// Sets the number of extension icons that should be visible.
// If count == size(), this will set the visible icon count to -1, meaning
// "show all actions".

Powered by Google App Engine
This is Rietveld 408576698