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

Unified Diff: chrome/browser/extensions/api/management/management_api.cc

Issue 238633009: cleanup: Use EventRouter instead of ExtensionSystem::Get->event_router() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 8 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/api/management/management_api.cc
diff --git a/chrome/browser/extensions/api/management/management_api.cc b/chrome/browser/extensions/api/management/management_api.cc
index c4473fd1d615210bc9408af43a46f5f1f202aebc..e34ba8e344114289361c714ffecfd624617bd3b2 100644
--- a/chrome/browser/extensions/api/management/management_api.cc
+++ b/chrome/browser/extensions/api/management/management_api.cc
@@ -714,13 +714,12 @@ void ManagementEventRouter::Observe(
}
scoped_ptr<Event> event(new Event(event_name, args.Pass()));
- ExtensionSystem::Get(profile)->event_router()->BroadcastEvent(event.Pass());
+ EventRouter::Get(profile)->BroadcastEvent(event.Pass());
}
ManagementAPI::ManagementAPI(content::BrowserContext* context)
: browser_context_(context) {
- EventRouter* event_router =
- ExtensionSystem::Get(browser_context_)->event_router();
+ EventRouter* event_router = EventRouter::Get(browser_context_);
event_router->RegisterObserver(this, management::OnInstalled::kEventName);
event_router->RegisterObserver(this, management::OnUninstalled::kEventName);
event_router->RegisterObserver(this, management::OnEnabled::kEventName);
@@ -731,8 +730,7 @@ ManagementAPI::~ManagementAPI() {
}
void ManagementAPI::Shutdown() {
- ExtensionSystem::Get(browser_context_)->event_router()->UnregisterObserver(
- this);
+ EventRouter::Get(browser_context_)->UnregisterObserver(this);
}
static base::LazyInstance<BrowserContextKeyedAPIFactory<ManagementAPI> >
@@ -747,8 +745,7 @@ ManagementAPI::GetFactoryInstance() {
void ManagementAPI::OnListenerAdded(const EventListenerInfo& details) {
management_event_router_.reset(
new ManagementEventRouter(Profile::FromBrowserContext(browser_context_)));
- ExtensionSystem::Get(browser_context_)->event_router()->UnregisterObserver(
- this);
+ EventRouter::Get(browser_context_)->UnregisterObserver(this);
}
} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698