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

Unified Diff: chrome/browser/extensions/api/runtime/runtime_api.h

Issue 212603020: Break extensions ProcessManager dependency on Runtime API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase (pm-observer) Created 6 years, 9 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
« no previous file with comments | « no previous file | chrome/browser/extensions/api/runtime/runtime_api.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/runtime/runtime_api.h
diff --git a/chrome/browser/extensions/api/runtime/runtime_api.h b/chrome/browser/extensions/api/runtime/runtime_api.h
index 6e65e1b704cefa195cd1a124d3ae0757c42b0401..57ea8aceb8b36b7d376366a85edc61ffb7f0fed0 100644
--- a/chrome/browser/extensions/api/runtime/runtime_api.h
+++ b/chrome/browser/extensions/api/runtime/runtime_api.h
@@ -12,6 +12,7 @@
#include "content/public/browser/notification_observer.h"
#include "content/public/browser/notification_registrar.h"
#include "extensions/browser/browser_context_keyed_api_factory.h"
+#include "extensions/browser/process_manager_observer.h"
#include "extensions/browser/update_observer.h"
class Profile;
@@ -33,7 +34,8 @@ class ExtensionHost;
// its related incognito instance.
class RuntimeAPI : public BrowserContextKeyedAPI,
public content::NotificationObserver,
- public extensions::UpdateObserver {
+ public UpdateObserver,
+ public ProcessManagerObserver {
public:
static BrowserContextKeyedAPIFactory<RuntimeAPI>* GetFactoryInstance();
@@ -57,11 +59,15 @@ class RuntimeAPI : public BrowserContextKeyedAPI,
static const char* service_name() { return "RuntimeAPI"; }
static const bool kServiceRedirectedInIncognito = true;
static const bool kServiceIsNULLWhileTesting = true;
+ virtual void Shutdown() OVERRIDE;
// extensions::UpdateObserver overrides:
virtual void OnAppUpdateAvailable(const Extension* extension) OVERRIDE;
virtual void OnChromeUpdateAvailable() OVERRIDE;
+ // ProcessManagerObserver implementation:
+ virtual void OnBackgroundHostStartup(const Extension* extension) OVERRIDE;
+
content::BrowserContext* browser_context_;
// True if we should dispatch the chrome.runtime.onInstalled event with
« no previous file with comments | « no previous file | chrome/browser/extensions/api/runtime/runtime_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698