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

Unified Diff: chrome/browser/profiles/off_the_record_profile_io_data.cc

Issue 2878973002: Remove some configuration from the extension URLRequestContexts. (Closed)
Patch Set: Pointless merge Created 3 years, 7 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_io_data.cc
diff --git a/chrome/browser/profiles/off_the_record_profile_io_data.cc b/chrome/browser/profiles/off_the_record_profile_io_data.cc
index d75015105e3d77c124255ff03f16aa3c7ef4e614..f9d8e9da023acf7de10d10dd56754773d68d7f0c 100644
--- a/chrome/browser/profiles/off_the_record_profile_io_data.cc
+++ b/chrome/browser/profiles/off_the_record_profile_io_data.cc
@@ -255,42 +255,15 @@ void OffTheRecordProfileIOData::InitializeInternal(
void OffTheRecordProfileIOData::
InitializeExtensionsRequestContext(ProfileParams* profile_params) const {
- net::URLRequestContext* extensions_context = extensions_request_context();
-
- IOThread* const io_thread = profile_params->io_thread;
- IOThread::Globals* const io_thread_globals = io_thread->globals();
-
- ApplyProfileParamsToContext(extensions_context);
-
- extensions_context->set_transport_security_state(transport_security_state());
-
- extensions_context->set_net_log(io_thread->net_log());
-
- extensions_context->set_cert_transparency_verifier(
- io_thread_globals->cert_transparency_verifier.get());
-
// All we care about for extensions is the cookie store. For incognito, we
// use a non-persistent cookie store.
+ net::URLRequestContext* extensions_context = extensions_request_context();
+
content::CookieStoreConfig cookie_config;
// Enable cookies for chrome-extension URLs.
cookie_config.cookieable_schemes.push_back(extensions::kExtensionScheme);
extensions_cookie_store_ = content::CreateCookieStore(cookie_config);
extensions_context->set_cookie_store(extensions_cookie_store_.get());
-
- std::unique_ptr<net::URLRequestJobFactoryImpl> extensions_job_factory(
- new net::URLRequestJobFactoryImpl());
- // TODO(shalev): The extensions_job_factory has a NULL NetworkDelegate.
- // Without a network_delegate, this protocol handler will never
- // handle file: requests, but as a side effect it makes
- // job_factory::IsHandledProtocol return true, which prevents attempts to
- // handle the protocol externally. We pass NULL in to
- // SetUpJobFactoryDefaults() to get this effect.
- extensions_job_factory_ = SetUpJobFactoryDefaults(
- std::move(extensions_job_factory),
- content::URLRequestInterceptorScopedVector(),
- std::unique_ptr<ProtocolHandlerRegistry::JobInterceptorFactory>(), NULL,
- io_thread_globals->host_resolver.get());
- extensions_context->set_job_factory(extensions_job_factory_.get());
}
net::URLRequestContext* OffTheRecordProfileIOData::InitializeAppRequestContext(
« no previous file with comments | « chrome/browser/profiles/off_the_record_profile_io_data.h ('k') | chrome/browser/profiles/profile_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698