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

Unified Diff: android_webview/browser/net/aw_url_request_context_getter.cc

Issue 2258823002: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 4 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: android_webview/browser/net/aw_url_request_context_getter.cc
diff --git a/android_webview/browser/net/aw_url_request_context_getter.cc b/android_webview/browser/net/aw_url_request_context_getter.cc
index a469cce13bb578ed114afe8b1340001f3ef77afb..7ad7c4c7bc648a1eb9927ad99ee878826dfdb0e3 100644
--- a/android_webview/browser/net/aw_url_request_context_getter.cc
+++ b/android_webview/browser/net/aw_url_request_context_getter.cc
@@ -102,13 +102,13 @@ std::unique_ptr<net::URLRequestJobFactory> CreateJobFactory(
// AwContentBrowserClient::IsHandledURL.
bool set_protocol = aw_job_factory->SetProtocolHandler(
url::kFileScheme,
- base::WrapUnique(new net::FileProtocolHandler(
+ base::MakeUnique<net::FileProtocolHandler>(
content::BrowserThread::GetBlockingPool()
->GetTaskRunnerWithShutdownBehavior(
- base::SequencedWorkerPool::SKIP_ON_SHUTDOWN))));
+ base::SequencedWorkerPool::SKIP_ON_SHUTDOWN)));
DCHECK(set_protocol);
set_protocol = aw_job_factory->SetProtocolHandler(
- url::kDataScheme, base::WrapUnique(new net::DataProtocolHandler()));
+ url::kDataScheme, base::MakeUnique<net::DataProtocolHandler>());
DCHECK(set_protocol);
set_protocol = aw_job_factory->SetProtocolHandler(
url::kBlobScheme,
@@ -205,7 +205,7 @@ void AwURLRequestContextGetter::InitializeURLRequestContext() {
AwBrowserContext* browser_context = AwBrowserContext::GetDefault();
DCHECK(browser_context);
- builder.set_network_delegate(base::WrapUnique(new AwNetworkDelegate()));
+ builder.set_network_delegate(base::MakeUnique<AwNetworkDelegate>());
#if !defined(DISABLE_FTP_SUPPORT)
builder.set_ftp_enabled(false); // Android WebView does not support ftp yet.
#endif
@@ -231,9 +231,8 @@ void AwURLRequestContextGetter::InitializeURLRequestContext() {
builder.set_proxy_service(net::ProxyService::CreateWithoutProxyResolver(
std::move(proxy_config_service_), net_log_.get()));
builder.set_net_log(net_log_.get());
- builder.SetCookieAndChannelIdStores(
- base::WrapUnique(new AwCookieStoreWrapper()),
- std::move(channel_id_service));
+ builder.SetCookieAndChannelIdStores(base::MakeUnique<AwCookieStoreWrapper>(),
+ std::move(channel_id_service));
net::URLRequestContextBuilder::HttpCacheParams cache_params;
cache_params.type =
« no previous file with comments | « android_webview/browser/net/aw_request_interceptor.cc ('k') | android_webview/browser/test/rendering_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698