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

Unified Diff: chrome/browser/io_thread.cc

Issue 2428143002: Clean up FtpTransactionFacory ownership. (Closed)
Patch Set: Oops 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: chrome/browser/io_thread.cc
diff --git a/chrome/browser/io_thread.cc b/chrome/browser/io_thread.cc
index c5c7ce179e4447693d969cfb3aae67c7f6c456b1..a6c94b737c602bfa31294d32d756685598243e70 100644
--- a/chrome/browser/io_thread.cc
+++ b/chrome/browser/io_thread.cc
@@ -83,7 +83,6 @@
#include "net/dns/host_cache.h"
#include "net/dns/host_resolver.h"
#include "net/dns/mapped_host_resolver.h"
-#include "net/ftp/ftp_network_layer.h"
#include "net/http/http_auth_filter.h"
#include "net/http/http_auth_handler_factory.h"
#include "net/http/http_auth_preferences.h"
@@ -1062,12 +1061,9 @@ net::URLRequestContext* IOThread::ConstructProxyScriptFetcherContext(
->GetTaskRunnerWithShutdownBehavior(
base::SequencedWorkerPool::SKIP_ON_SHUTDOWN)));
#if !defined(DISABLE_FTP_SUPPORT)
- globals->proxy_script_fetcher_ftp_transaction_factory.reset(
- new net::FtpNetworkLayer(globals->host_resolver.get()));
job_factory->SetProtocolHandler(
url::kFtpScheme,
- base::MakeUnique<net::FtpProtocolHandler>(
- globals->proxy_script_fetcher_ftp_transaction_factory.get()));
+ net::FtpProtocolHandler::Create(globals->host_resolver.get()));
#endif
globals->proxy_script_fetcher_url_request_job_factory =
std::move(job_factory);

Powered by Google App Engine
This is Rietveld 408576698