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

Unified Diff: chrome/browser/renderer_host/chrome_render_message_filter.cc

Issue 9369013: Take extensions out of Profile into a profile-keyed service, ExtensionSystem. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 10 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/renderer_host/chrome_render_message_filter.cc
diff --git a/chrome/browser/renderer_host/chrome_render_message_filter.cc b/chrome/browser/renderer_host/chrome_render_message_filter.cc
index d4e5c160874af1d55653e07ddd1fa670977fce44..e8cc3586595529f84cd28a0cd3e69a6576b6b417 100644
--- a/chrome/browser/renderer_host/chrome_render_message_filter.cc
+++ b/chrome/browser/renderer_host/chrome_render_message_filter.cc
@@ -17,6 +17,8 @@
#include "chrome/browser/extensions/extension_info_map.h"
#include "chrome/browser/extensions/extension_message_service.h"
#include "chrome/browser/extensions/extension_process_manager.h"
+#include "chrome/browser/extensions/extension_system.h"
+#include "chrome/browser/extensions/extension_system_factory.h"
#include "chrome/browser/metrics/histogram_synchronizer.h"
#include "chrome/browser/nacl_host/nacl_process_host.h"
#include "chrome/browser/net/chrome_url_request_context.h"
@@ -54,7 +56,8 @@ ChromeRenderMessageFilter::ChromeRenderMessageFilter(
: render_process_id_(render_process_id),
profile_(profile),
request_context_(request_context),
- extension_info_map_(profile->GetExtensionInfoMap()),
+ extension_info_map_(
+ ExtensionSystemFactory::GetForProfile(profile)->info_map()),
cookie_settings_(CookieSettings::GetForProfile(profile)),
resource_context_(profile->GetResourceContext()),
weak_ptr_factory_(ALLOW_THIS_IN_INITIALIZER_LIST(this)) {
@@ -267,9 +270,10 @@ void ChromeRenderMessageFilter::OpenChannelToExtensionOnUIThread(
const std::string& target_extension_id,
const std::string& channel_name) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- profile_->GetExtensionMessageService()->OpenChannelToExtension(
- source_process_id, source_routing_id, receiver_port_id,
- source_extension_id, target_extension_id, channel_name);
+ ExtensionSystemFactory::GetForProfile(profile_)->message_service()->
+ OpenChannelToExtension(
+ source_process_id, source_routing_id, receiver_port_id,
+ source_extension_id, target_extension_id, channel_name);
}
void ChromeRenderMessageFilter::OnOpenChannelToTab(
@@ -292,7 +296,8 @@ void ChromeRenderMessageFilter::OpenChannelToTabOnUIThread(
const std::string& extension_id,
const std::string& channel_name) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
- profile_->GetExtensionMessageService()->OpenChannelToTab(
+ ExtensionSystemFactory::GetForProfile(profile_)->message_service()->
+ OpenChannelToTab(
source_process_id, source_routing_id, receiver_port_id,
tab_id, extension_id, channel_name);
}
@@ -387,8 +392,10 @@ void ChromeRenderMessageFilter::OnExtensionCloseChannel(int port_id) {
if (!content::RenderProcessHost::FromID(render_process_id_))
return; // To guard against crash in browser_tests shutdown.
- if (profile_->GetExtensionMessageService())
- profile_->GetExtensionMessageService()->CloseChannel(port_id);
+ ExtensionMessageService* message_service =
+ ExtensionSystemFactory::GetForProfile(profile_)->message_service();
+ if (message_service)
+ message_service->CloseChannel(port_id);
}
void ChromeRenderMessageFilter::OnExtensionRequestForIOThread(

Powered by Google App Engine
This is Rietveld 408576698