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

Unified Diff: chrome/browser/profiles/off_the_record_profile_impl.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: rerebase 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/profiles/off_the_record_profile_impl.cc
diff --git a/chrome/browser/profiles/off_the_record_profile_impl.cc b/chrome/browser/profiles/off_the_record_profile_impl.cc
index 480eb1b43293d48d0360cc1db2ecdeb99bc66715..8b0a8e7601d8f7c1e21ab8a39aaedb19cb8a71c1 100644
--- a/chrome/browser/profiles/off_the_record_profile_impl.cc
+++ b/chrome/browser/profiles/off_the_record_profile_impl.cc
@@ -21,11 +21,13 @@
#include "chrome/browser/download/download_service_factory.h"
#include "chrome/browser/extensions/api/webrequest/webrequest_api.h"
#include "chrome/browser/extensions/extension_info_map.h"
-#include "chrome/browser/extensions/extension_message_service.h"
#include "chrome/browser/extensions/extension_pref_store.h"
+#include "chrome/browser/extensions/extension_pref_value_map.h"
#include "chrome/browser/extensions/extension_process_manager.h"
#include "chrome/browser/extensions/extension_service.h"
#include "chrome/browser/extensions/extension_special_storage_policy.h"
+#include "chrome/browser/extensions/extension_system.h"
+#include "chrome/browser/extensions/extension_system_factory.h"
#include "chrome/browser/io_thread.h"
#include "chrome/browser/net/proxy_service_factory.h"
#include "chrome/browser/plugin_prefs.h"
@@ -85,8 +87,6 @@ OffTheRecordProfileImpl::OffTheRecordProfileImpl(Profile* real_profile)
}
void OffTheRecordProfileImpl::Init() {
- extension_process_manager_.reset(ExtensionProcessManager::Create(this));
-
BrowserList::AddObserver(this);
ProfileDependencyManager::GetInstance()->CreateProfileServices(this, false);
@@ -124,6 +124,13 @@ OffTheRecordProfileImpl::~OffTheRecordProfileImpl() {
ChromePluginServiceFilter::GetInstance()->UnregisterResourceContext(
io_data_.GetResourceContextNoInit());
+ ExtensionService* extension_service =
+ ExtensionSystemFactory::GetForProfile(this)->extension_service();
+ if (extension_service) {
+ extension_service->extension_prefs()->
+ ClearIncognitoSessionOnlyContentSettings();
+ }
+
ProfileDependencyManager::GetInstance()->DestroyProfileServices(this);
BrowserThread::PostTask(
@@ -138,12 +145,6 @@ OffTheRecordProfileImpl::~OffTheRecordProfileImpl() {
if (pref_proxy_config_tracker_.get())
pref_proxy_config_tracker_->DetachFromPrefService();
- ExtensionService* extension_service = GetExtensionService();
- if (extension_service) {
- ExtensionPrefs* extension_prefs = extension_service->extension_prefs();
- extension_prefs->ClearIncognitoSessionOnlyContentSettings();
- }
-
// Clears any data the network stack contains that may be related to the
// OTR session.
g_browser_process->io_thread()->ChangedToOnTheRecord();
@@ -196,33 +197,25 @@ VisitedLinkMaster* OffTheRecordProfileImpl::GetVisitedLinkMaster() {
return NULL;
}
-ExtensionService* OffTheRecordProfileImpl::GetExtensionService() {
- return GetOriginalProfile()->GetExtensionService();
+ExtensionPrefValueMap* OffTheRecordProfileImpl::GetExtensionPrefValueMap() {
+ return NULL;
}
-UserScriptMaster* OffTheRecordProfileImpl::GetUserScriptMaster() {
- return GetOriginalProfile()->GetUserScriptMaster();
+ExtensionService* OffTheRecordProfileImpl::GetExtensionService() {
+ return ExtensionSystemFactory::GetForProfile(this)->extension_service();
}
-ExtensionDevToolsManager*
- OffTheRecordProfileImpl::GetExtensionDevToolsManager() {
- // TODO(mpcomplete): figure out whether we should return the original
- // profile's version.
- return NULL;
+UserScriptMaster* OffTheRecordProfileImpl::GetUserScriptMaster() {
+ return ExtensionSystemFactory::GetForProfile(this)->user_script_master();
}
ExtensionProcessManager*
OffTheRecordProfileImpl::GetExtensionProcessManager() {
- return extension_process_manager_.get();
-}
-
-ExtensionMessageService*
- OffTheRecordProfileImpl::GetExtensionMessageService() {
- return GetOriginalProfile()->GetExtensionMessageService();
+ return ExtensionSystemFactory::GetForProfile(this)->process_manager();
}
ExtensionEventRouter* OffTheRecordProfileImpl::GetExtensionEventRouter() {
- return GetOriginalProfile()->GetExtensionEventRouter();
+ return ExtensionSystemFactory::GetForProfile(this)->event_router();
}
ExtensionSpecialStoragePolicy*
@@ -411,10 +404,6 @@ history::TopSites* OffTheRecordProfileImpl::GetTopSites() {
void OffTheRecordProfileImpl::MarkAsCleanShutdown() {
}
-void OffTheRecordProfileImpl::InitExtensions(bool extensions_enabled) {
- NOTREACHED();
-}
-
void OffTheRecordProfileImpl::InitPromoResources() {
NOTREACHED();
}
@@ -453,10 +442,6 @@ void OffTheRecordProfileImpl::OnBrowserAdded(const Browser* browser) {
void OffTheRecordProfileImpl::OnBrowserRemoved(const Browser* browser) {
}
-ExtensionInfoMap* OffTheRecordProfileImpl::GetExtensionInfoMap() {
- return profile_->GetExtensionInfoMap();
-}
-
ChromeURLDataManager* OffTheRecordProfileImpl::GetChromeURLDataManager() {
if (!chrome_url_data_manager_.get())
chrome_url_data_manager_.reset(new ChromeURLDataManager(
« no previous file with comments | « chrome/browser/profiles/off_the_record_profile_impl.h ('k') | chrome/browser/profiles/off_the_record_profile_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698