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

Side by Side 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: fix compile error for chromeos build. 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/extensions/api/management/management_api.h" 5 #include "chrome/browser/extensions/api/management/management_api.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 696 matching lines...) Expand 10 before | Expand all | Expand 10 after
707 scoped_ptr<base::ListValue> args(new base::ListValue()); 707 scoped_ptr<base::ListValue> args(new base::ListValue());
708 if (event_name == management::OnUninstalled::kEventName) { 708 if (event_name == management::OnUninstalled::kEventName) {
709 args->Append(new base::StringValue(extension->id())); 709 args->Append(new base::StringValue(extension->id()));
710 } else { 710 } else {
711 scoped_ptr<management::ExtensionInfo> info = CreateExtensionInfo( 711 scoped_ptr<management::ExtensionInfo> info = CreateExtensionInfo(
712 *extension, ExtensionSystem::Get(profile)); 712 *extension, ExtensionSystem::Get(profile));
713 args->Append(info->ToValue().release()); 713 args->Append(info->ToValue().release());
714 } 714 }
715 715
716 scoped_ptr<Event> event(new Event(event_name, args.Pass())); 716 scoped_ptr<Event> event(new Event(event_name, args.Pass()));
717 ExtensionSystem::Get(profile)->event_router()->BroadcastEvent(event.Pass()); 717 EventRouter::Get(profile)->BroadcastEvent(event.Pass());
718 } 718 }
719 719
720 ManagementAPI::ManagementAPI(content::BrowserContext* context) 720 ManagementAPI::ManagementAPI(content::BrowserContext* context)
721 : browser_context_(context) { 721 : browser_context_(context) {
722 EventRouter* event_router = 722 EventRouter* event_router = EventRouter::Get(browser_context_);
723 ExtensionSystem::Get(browser_context_)->event_router();
724 event_router->RegisterObserver(this, management::OnInstalled::kEventName); 723 event_router->RegisterObserver(this, management::OnInstalled::kEventName);
725 event_router->RegisterObserver(this, management::OnUninstalled::kEventName); 724 event_router->RegisterObserver(this, management::OnUninstalled::kEventName);
726 event_router->RegisterObserver(this, management::OnEnabled::kEventName); 725 event_router->RegisterObserver(this, management::OnEnabled::kEventName);
727 event_router->RegisterObserver(this, management::OnDisabled::kEventName); 726 event_router->RegisterObserver(this, management::OnDisabled::kEventName);
728 } 727 }
729 728
730 ManagementAPI::~ManagementAPI() { 729 ManagementAPI::~ManagementAPI() {
731 } 730 }
732 731
733 void ManagementAPI::Shutdown() { 732 void ManagementAPI::Shutdown() {
734 ExtensionSystem::Get(browser_context_)->event_router()->UnregisterObserver( 733 EventRouter::Get(browser_context_)->UnregisterObserver(this);
735 this);
736 } 734 }
737 735
738 static base::LazyInstance<BrowserContextKeyedAPIFactory<ManagementAPI> > 736 static base::LazyInstance<BrowserContextKeyedAPIFactory<ManagementAPI> >
739 g_factory = LAZY_INSTANCE_INITIALIZER; 737 g_factory = LAZY_INSTANCE_INITIALIZER;
740 738
741 // static 739 // static
742 BrowserContextKeyedAPIFactory<ManagementAPI>* 740 BrowserContextKeyedAPIFactory<ManagementAPI>*
743 ManagementAPI::GetFactoryInstance() { 741 ManagementAPI::GetFactoryInstance() {
744 return g_factory.Pointer(); 742 return g_factory.Pointer();
745 } 743 }
746 744
747 void ManagementAPI::OnListenerAdded(const EventListenerInfo& details) { 745 void ManagementAPI::OnListenerAdded(const EventListenerInfo& details) {
748 management_event_router_.reset( 746 management_event_router_.reset(
749 new ManagementEventRouter(Profile::FromBrowserContext(browser_context_))); 747 new ManagementEventRouter(Profile::FromBrowserContext(browser_context_)));
750 ExtensionSystem::Get(browser_context_)->event_router()->UnregisterObserver( 748 EventRouter::Get(browser_context_)->UnregisterObserver(this);
751 this);
752 } 749 }
753 750
754 } // namespace extensions 751 } // namespace extensions
OLDNEW
« no previous file with comments | « chrome/browser/extensions/api/log_private/log_private_api_chromeos.cc ('k') | chrome/browser/extensions/api/mdns/mdns_api.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698