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

Side by Side Diff: net/socket/ssl_client_socket_pool.cc

Issue 8526006: Close idle sockets next time we are about to send data. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: code conventions Created 9 years, 1 month 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) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/socket/ssl_client_socket_pool.h" 5 #include "net/socket/ssl_client_socket_pool.h"
6 6
7 #include "base/metrics/field_trial.h" 7 #include "base/metrics/field_trial.h"
8 #include "base/metrics/histogram.h" 8 #include "base/metrics/histogram.h"
9 #include "base/values.h" 9 #include "base/values.h"
10 #include "net/base/net_errors.h" 10 #include "net/base/net_errors.h"
(...skipping 436 matching lines...) Expand 10 before | Expand all | Expand 10 after
447 CertVerifier* cert_verifier, 447 CertVerifier* cert_verifier,
448 OriginBoundCertService* origin_bound_cert_service, 448 OriginBoundCertService* origin_bound_cert_service,
449 DnsRRResolver* dnsrr_resolver, 449 DnsRRResolver* dnsrr_resolver,
450 DnsCertProvenanceChecker* dns_cert_checker, 450 DnsCertProvenanceChecker* dns_cert_checker,
451 SSLHostInfoFactory* ssl_host_info_factory, 451 SSLHostInfoFactory* ssl_host_info_factory,
452 ClientSocketFactory* client_socket_factory, 452 ClientSocketFactory* client_socket_factory,
453 TransportClientSocketPool* transport_pool, 453 TransportClientSocketPool* transport_pool,
454 SOCKSClientSocketPool* socks_pool, 454 SOCKSClientSocketPool* socks_pool,
455 HttpProxyClientSocketPool* http_proxy_pool, 455 HttpProxyClientSocketPool* http_proxy_pool,
456 SSLConfigService* ssl_config_service, 456 SSLConfigService* ssl_config_service,
457 NetLog* net_log) 457 NetLog* net_log,
458 const ClientSocketPoolOptions& options)
458 : transport_pool_(transport_pool), 459 : transport_pool_(transport_pool),
459 socks_pool_(socks_pool), 460 socks_pool_(socks_pool),
460 http_proxy_pool_(http_proxy_pool), 461 http_proxy_pool_(http_proxy_pool),
461 base_(max_sockets, max_sockets_per_group, histograms, 462 base_(max_sockets, max_sockets_per_group, histograms,
462 ClientSocketPool::unused_idle_socket_timeout(), 463 ClientSocketPool::unused_idle_socket_timeout(),
463 ClientSocketPool::used_idle_socket_timeout(), 464 ClientSocketPool::used_idle_socket_timeout(),
464 new SSLConnectJobFactory(transport_pool, 465 new SSLConnectJobFactory(transport_pool,
465 socks_pool, 466 socks_pool,
466 http_proxy_pool, 467 http_proxy_pool,
467 client_socket_factory, 468 client_socket_factory,
468 host_resolver, 469 host_resolver,
469 SSLClientSocketContext( 470 SSLClientSocketContext(
470 cert_verifier, 471 cert_verifier,
471 origin_bound_cert_service, 472 origin_bound_cert_service,
472 dnsrr_resolver, 473 dnsrr_resolver,
473 dns_cert_checker, 474 dns_cert_checker,
474 ssl_host_info_factory), 475 ssl_host_info_factory),
475 net_log)), 476 net_log),
477 options),
476 ssl_config_service_(ssl_config_service) { 478 ssl_config_service_(ssl_config_service) {
477 if (ssl_config_service_) 479 if (ssl_config_service_)
478 ssl_config_service_->AddObserver(this); 480 ssl_config_service_->AddObserver(this);
479 } 481 }
480 482
481 SSLClientSocketPool::~SSLClientSocketPool() { 483 SSLClientSocketPool::~SSLClientSocketPool() {
482 if (ssl_config_service_) 484 if (ssl_config_service_)
483 ssl_config_service_->RemoveObserver(this); 485 ssl_config_service_->RemoveObserver(this);
484 } 486 }
485 487
(...skipping 96 matching lines...) Expand 10 before | Expand all | Expand 10 after
582 584
583 ClientSocketPoolHistograms* SSLClientSocketPool::histograms() const { 585 ClientSocketPoolHistograms* SSLClientSocketPool::histograms() const {
584 return base_.histograms(); 586 return base_.histograms();
585 } 587 }
586 588
587 void SSLClientSocketPool::OnSSLConfigChanged() { 589 void SSLClientSocketPool::OnSSLConfigChanged() {
588 Flush(); 590 Flush();
589 } 591 }
590 592
591 } // namespace net 593 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698