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

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

Issue 496403003: Remove NOTIFICATION_EXTENSION_PAGE_ACTIONS_UPDATED (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase 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/page_action_controller.h
diff --git a/chrome/browser/extensions/page_action_controller.h b/chrome/browser/extensions/page_action_controller.h
index 93560ce967d73f0acad270629200eb6f87c9c8e4..08325a65543ca091d7f909caf5869bd5b4fcd290 100644
--- a/chrome/browser/extensions/page_action_controller.h
+++ b/chrome/browser/extensions/page_action_controller.h
@@ -5,21 +5,21 @@
#ifndef CHROME_BROWSER_EXTENSIONS_PAGE_ACTION_CONTROLLER_H_
#define CHROME_BROWSER_EXTENSIONS_PAGE_ACTION_CONTROLLER_H_
-#include <string>
-
-#include "chrome/browser/extensions/api/extension_action/extension_action_api.h"
+#include "base/macros.h"
#include "chrome/browser/extensions/location_bar_controller.h"
-class Profile;
+namespace content {
+class BrowserContext;
+class WebContents;
+}
namespace extensions {
-class ExtensionRegistry;
+class Extension;
// A LocationBarControllerProvider which populates the location bar with icons
// based on the page_action extension API.
// TODO(rdevlin.cronin): This isn't really a controller.
-class PageActionController : public LocationBarController::ActionProvider,
- public ExtensionActionAPI::Observer {
+class PageActionController : public LocationBarController::ActionProvider {
public:
explicit PageActionController(content::WebContents* web_contents);
virtual ~PageActionController();
@@ -32,20 +32,11 @@ class PageActionController : public LocationBarController::ActionProvider,
virtual void OnNavigated() OVERRIDE;
private:
- // ExtensionActionAPI::Observer implementation.
- virtual void OnExtensionActionUpdated(
- ExtensionAction* extension_action,
- content::WebContents* web_contents,
- content::BrowserContext* browser_context) OVERRIDE;
-
- // Returns the associated Profile.
- Profile* GetProfile();
-
// The associated WebContents.
content::WebContents* web_contents_;
- ScopedObserver<ExtensionActionAPI, ExtensionActionAPI::Observer>
- extension_action_observer_;
+ // The associated browser context.
+ content::BrowserContext* browser_context_;
DISALLOW_COPY_AND_ASSIGN(PageActionController);
};

Powered by Google App Engine
This is Rietveld 408576698