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

Side by Side Diff: net/url_request/url_request_context_builder.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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/url_request/url_request_context_builder.h" 5 #include "net/url_request/url_request_context_builder.h"
6 6
7 #include <string> 7 #include <string>
8 #include <utility> 8 #include <utility>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 239 matching lines...) Expand 10 before | Expand all | Expand 10 after
250 scoped_ptr<CertVerifier> cert_verifier) { 250 scoped_ptr<CertVerifier> cert_verifier) {
251 cert_verifier_ = std::move(cert_verifier); 251 cert_verifier_ = std::move(cert_verifier);
252 } 252 }
253 253
254 void URLRequestContextBuilder::SetInterceptors( 254 void URLRequestContextBuilder::SetInterceptors(
255 std::vector<scoped_ptr<URLRequestInterceptor>> url_request_interceptors) { 255 std::vector<scoped_ptr<URLRequestInterceptor>> url_request_interceptors) {
256 url_request_interceptors_ = std::move(url_request_interceptors); 256 url_request_interceptors_ = std::move(url_request_interceptors);
257 } 257 }
258 258
259 void URLRequestContextBuilder::SetCookieAndChannelIdStores( 259 void URLRequestContextBuilder::SetCookieAndChannelIdStores(
260 const scoped_refptr<CookieStore>& cookie_store, 260 scoped_ptr<CookieStore> cookie_store,
261 scoped_ptr<ChannelIDService> channel_id_service) { 261 scoped_ptr<ChannelIDService> channel_id_service) {
262 DCHECK(cookie_store); 262 DCHECK(cookie_store);
263 cookie_store_ = cookie_store; 263 cookie_store_ = std::move(cookie_store);
264 channel_id_service_ = std::move(channel_id_service); 264 channel_id_service_ = std::move(channel_id_service);
265 } 265 }
266 266
267 void URLRequestContextBuilder::SetFileTaskRunner( 267 void URLRequestContextBuilder::SetFileTaskRunner(
268 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner) { 268 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner) {
269 file_task_runner_ = task_runner; 269 file_task_runner_ = task_runner;
270 } 270 }
271 271
272 void URLRequestContextBuilder::SetProtocolHandler( 272 void URLRequestContextBuilder::SetProtocolHandler(
273 const std::string& scheme, 273 const std::string& scheme,
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
331 storage->set_ssl_config_service(new SSLConfigServiceDefaults); 331 storage->set_ssl_config_service(new SSLConfigServiceDefaults);
332 332
333 if (!http_auth_handler_factory_) { 333 if (!http_auth_handler_factory_) {
334 http_auth_handler_factory_ = 334 http_auth_handler_factory_ =
335 HttpAuthHandlerRegistryFactory::CreateDefault(context->host_resolver()); 335 HttpAuthHandlerRegistryFactory::CreateDefault(context->host_resolver());
336 } 336 }
337 337
338 storage->set_http_auth_handler_factory(std::move(http_auth_handler_factory_)); 338 storage->set_http_auth_handler_factory(std::move(http_auth_handler_factory_));
339 339
340 if (cookie_store_) { 340 if (cookie_store_) {
341 storage->set_cookie_store(cookie_store_.get()); 341 storage->set_cookie_store(std::move(cookie_store_));
342 storage->set_channel_id_service(std::move(channel_id_service_)); 342 storage->set_channel_id_service(std::move(channel_id_service_));
343 } else { 343 } else {
344 storage->set_cookie_store(new CookieMonster(NULL, NULL)); 344 storage->set_cookie_store(
345 make_scoped_ptr(new CookieMonster(nullptr, nullptr)));
345 // TODO(mmenke): This always creates a file thread, even when it ends up 346 // TODO(mmenke): This always creates a file thread, even when it ends up
346 // not being used. Consider lazily creating the thread. 347 // not being used. Consider lazily creating the thread.
347 storage->set_channel_id_service(make_scoped_ptr(new ChannelIDService( 348 storage->set_channel_id_service(make_scoped_ptr(new ChannelIDService(
348 new DefaultChannelIDStore(NULL), context->GetFileTaskRunner()))); 349 new DefaultChannelIDStore(NULL), context->GetFileTaskRunner())));
349 } 350 }
350 351
351 if (sdch_enabled_) { 352 if (sdch_enabled_) {
352 storage->set_sdch_manager(scoped_ptr<net::SdchManager>(new SdchManager())); 353 storage->set_sdch_manager(scoped_ptr<net::SdchManager>(new SdchManager()));
353 } 354 }
354 355
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
505 } 506 }
506 url_request_interceptors_.clear(); 507 url_request_interceptors_.clear();
507 } 508 }
508 storage->set_job_factory(std::move(top_job_factory)); 509 storage->set_job_factory(std::move(top_job_factory));
509 // TODO(willchan): Support sdch. 510 // TODO(willchan): Support sdch.
510 511
511 return std::move(context); 512 return std::move(context);
512 } 513 }
513 514
514 } // namespace net 515 } // namespace net
OLDNEW
« no previous file with comments | « net/url_request/url_request_context_builder.h ('k') | net/url_request/url_request_context_getter.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698