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

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

Issue 1701063002: CookieStore: Remove reference counting. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@threadsafe
Patch Set: merge Created 4 years, 9 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 08ef8dfd319162fe45ae9a0c082ab5e10a54e31b..726883f7cb18b2cfa6b4d2f1326fd5ea93db5017 100644
--- a/android_webview/browser/net/aw_url_request_context_getter.cc
+++ b/android_webview/browser/net/aw_url_request_context_getter.cc
@@ -200,8 +200,6 @@ void AwURLRequestContextGetter::InitializeURLRequestContext() {
DCHECK_CURRENTLY_ON(BrowserThread::IO);
DCHECK(!url_request_context_);
- cookie_store_ = new AwCookieStoreWrapper();
-
net::URLRequestContextBuilder builder;
scoped_ptr<AwNetworkDelegate> aw_network_delegate(new AwNetworkDelegate());
@@ -237,8 +235,9 @@ 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(cookie_store_,
- std::move(channel_id_service));
+ builder.SetCookieAndChannelIdStores(
+ make_scoped_ptr(new AwCookieStoreWrapper()),
+ std::move(channel_id_service));
net::URLRequestContextBuilder::HttpCacheParams cache_params;
cache_params.type =
« no previous file with comments | « android_webview/browser/net/aw_url_request_context_getter.h ('k') | android_webview/native/cookie_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698