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

Unified Diff: components/cronet/ios/cronet_environment.cc

Issue 1888963004: Add HttpProtocolHandler and convert everything to use it (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-supports-scheme
Patch Set: rebase (needs fixing) Created 4 years, 2 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: components/cronet/ios/cronet_environment.cc
diff --git a/components/cronet/ios/cronet_environment.cc b/components/cronet/ios/cronet_environment.cc
index cb65294ae6848fb8db178a64976d3afd92faf135..174f3b60168cc502807d6a5c131ae063ad11f2ef 100644
--- a/components/cronet/ios/cronet_environment.cc
+++ b/components/cronet/ios/cronet_environment.cc
@@ -343,9 +343,10 @@ void CronetEnvironment::InitializeOnNetworkThread() {
true /* set_up_quic_server_info */);
main_context_->set_http_transaction_factory(main_cache);
- net::URLRequestJobFactoryImpl* job_factory =
- new net::URLRequestJobFactoryImpl;
- main_context_->set_job_factory(job_factory);
+ // TODO(mgersh): Fix this leak too. See https://crbug.com/523858.
+ std::unique_ptr<net::URLRequestJobFactoryImpl> job_factory =
+ net::URLRequestJobFactoryImpl::CreateWithHttpProtocolHandlers();
+ main_context_->set_job_factory(job_factory.release());
main_context_->set_net_log(net_log_.get());
}

Powered by Google App Engine
This is Rietveld 408576698