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

Unified Diff: net/url_request/url_request_context_builder.cc

Issue 2259823002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 3 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: net/url_request/url_request_context_builder.cc
diff --git a/net/url_request/url_request_context_builder.cc b/net/url_request/url_request_context_builder.cc
index 4f3c3a9cdf60c5f24d02822c7942841429962124..4da1a7324a21cc73b0ce05a8c2e50427a1a99451 100644
--- a/net/url_request/url_request_context_builder.cc
+++ b/net/url_request/url_request_context_builder.cc
@@ -304,8 +304,9 @@ std::unique_ptr<URLRequestContext> URLRequestContextBuilder::Build() {
new ContainerURLRequestContext(file_task_runner_));
URLRequestContextStorage* storage = context->storage();
- storage->set_http_user_agent_settings(base::WrapUnique(
- new StaticHttpUserAgentSettings(accept_language_, user_agent_)));
+ storage->set_http_user_agent_settings(
+ base::MakeUnique<StaticHttpUserAgentSettings>(accept_language_,
+ user_agent_));
if (!network_delegate_)
network_delegate_.reset(new BasicNetworkDelegate);
@@ -371,7 +372,7 @@ std::unique_ptr<URLRequestContext> URLRequestContextBuilder::Build() {
}
storage->set_transport_security_state(
- base::WrapUnique(new TransportSecurityState()));
+ base::MakeUnique<TransportSecurityState>());
if (!transport_security_persister_path_.empty()) {
context->set_transport_security_persister(
base::WrapUnique<TransportSecurityPersister>(
@@ -406,7 +407,7 @@ std::unique_ptr<URLRequestContext> URLRequestContextBuilder::Build() {
if (throttling_enabled_) {
storage->set_throttler_manager(
- base::WrapUnique(new URLRequestThrottlerManager()));
+ base::MakeUnique<URLRequestThrottlerManager>());
}
HttpNetworkSession::Params network_session_params;
@@ -461,7 +462,7 @@ std::unique_ptr<URLRequestContext> URLRequestContextBuilder::Build() {
}
storage->set_http_network_session(
- base::WrapUnique(new HttpNetworkSession(network_session_params)));
+ base::MakeUnique<HttpNetworkSession>(network_session_params));
std::unique_ptr<HttpTransactionFactory> http_transaction_factory;
if (http_cache_enabled_) {
@@ -503,8 +504,8 @@ std::unique_ptr<URLRequestContext> URLRequestContextBuilder::Build() {
#if !defined(DISABLE_FILE_SUPPORT)
if (file_enabled_) {
job_factory->SetProtocolHandler(
- "file", base::WrapUnique(
- new FileProtocolHandler(context->GetFileTaskRunner())));
+ "file",
+ base::MakeUnique<FileProtocolHandler>(context->GetFileTaskRunner()));
}
#endif // !defined(DISABLE_FILE_SUPPORT)
@@ -512,9 +513,8 @@ std::unique_ptr<URLRequestContext> URLRequestContextBuilder::Build() {
if (ftp_enabled_) {
ftp_transaction_factory_.reset(
new FtpNetworkLayer(context->host_resolver()));
- job_factory->SetProtocolHandler(
- "ftp", base::WrapUnique(
- new FtpProtocolHandler(ftp_transaction_factory_.get())));
+ job_factory->SetProtocolHandler("ftp", base::MakeUnique<FtpProtocolHandler>(
+ ftp_transaction_factory_.get()));
}
#endif // !defined(DISABLE_FTP_SUPPORT)
« no previous file with comments | « net/url_request/url_fetcher_impl_unittest.cc ('k') | net/url_request/url_request_context_builder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698