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

Unified Diff: net/http/http_proxy_client_socket_pool_spdy2_unittest.cc

Issue 9476035: Make CertVerifier a pure virtual interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Win shared fix Created 8 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: net/http/http_proxy_client_socket_pool_spdy2_unittest.cc
===================================================================
--- net/http/http_proxy_client_socket_pool_spdy2_unittest.cc (revision 128056)
+++ net/http/http_proxy_client_socket_pool_spdy2_unittest.cc (working copy)
@@ -63,12 +63,13 @@
&tcp_histograms_,
&socket_factory_),
ssl_histograms_("MockSSL"),
+ cert_verifier_(CertVerifier::CreateDefault()),
proxy_service_(ProxyService::CreateDirect()),
ssl_config_service_(new SSLConfigServiceDefaults),
ssl_socket_pool_(kMaxSockets, kMaxSocketsPerGroup,
&ssl_histograms_,
&host_resolver_,
- &cert_verifier_,
+ cert_verifier_.get(),
NULL /* server_bound_cert_store */,
NULL /* transport_security_state */,
NULL /* ssl_host_info_factory */,
@@ -181,7 +182,7 @@
HttpNetworkSession* CreateNetworkSession() {
HttpNetworkSession::Params params;
params.host_resolver = &host_resolver_;
- params.cert_verifier = &cert_verifier_;
+ params.cert_verifier = cert_verifier_.get();
params.proxy_service = proxy_service_.get();
params.client_socket_factory = &socket_factory_;
params.ssl_config_service = ssl_config_service_;
@@ -200,7 +201,7 @@
MockTransportClientSocketPool transport_socket_pool_;
ClientSocketPoolHistograms ssl_histograms_;
MockHostResolver host_resolver_;
- CertVerifier cert_verifier_;
+ scoped_ptr<CertVerifier> cert_verifier_;
const scoped_ptr<ProxyService> proxy_service_;
const scoped_refptr<SSLConfigService> ssl_config_service_;
SSLClientSocketPool ssl_socket_pool_;
« no previous file with comments | « net/http/http_proxy_client_socket_pool_spdy21_unittest.cc ('k') | net/http/http_proxy_client_socket_pool_spdy3_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698