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

Unified Diff: net/http/http_response_body_drainer_unittest.cc

Issue 2067843003: Require a CTVerifier and CTPolicyEnforcer for TLS/QUIC sockets (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixup Created 4 years, 6 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
« no previous file with comments | « net/http/http_network_transaction_ssl_unittest.cc ('k') | net/http/http_stream_factory_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_response_body_drainer_unittest.cc
diff --git a/net/http/http_response_body_drainer_unittest.cc b/net/http/http_response_body_drainer_unittest.cc
index fdaa3155853ea51981116246f52f7ac45b0be9ba..920e14442bcc36c2db955290356b08e81df1b5a2 100644
--- a/net/http/http_response_body_drainer_unittest.cc
+++ b/net/http/http_response_body_drainer_unittest.cc
@@ -19,6 +19,9 @@
#include "net/base/io_buffer.h"
#include "net/base/net_errors.h"
#include "net/base/test_completion_callback.h"
+#include "net/cert/ct_policy_enforcer.h"
+#include "net/cert/mock_cert_verifier.h"
+#include "net/cert/multi_log_ct_verifier.h"
#include "net/http/http_network_session.h"
#include "net/http/http_server_properties_impl.h"
#include "net/http/http_stream.h"
@@ -230,26 +233,31 @@ class HttpResponseBodyDrainerTest : public testing::Test {
: proxy_service_(ProxyService::CreateDirect()),
ssl_config_service_(new SSLConfigServiceDefaults),
http_server_properties_(new HttpServerPropertiesImpl()),
- transport_security_state_(new TransportSecurityState()),
session_(CreateNetworkSession()),
mock_stream_(new MockHttpStream(&result_waiter_)),
drainer_(new HttpResponseBodyDrainer(mock_stream_)) {}
~HttpResponseBodyDrainerTest() override {}
- HttpNetworkSession* CreateNetworkSession() const {
+ HttpNetworkSession* CreateNetworkSession() {
HttpNetworkSession::Params params;
params.proxy_service = proxy_service_.get();
params.ssl_config_service = ssl_config_service_.get();
params.http_server_properties = http_server_properties_.get();
- params.transport_security_state = transport_security_state_.get();
+ params.cert_verifier = &cert_verifier_;
+ params.transport_security_state = &transport_security_state_;
+ params.cert_transparency_verifier = &ct_verifier_;
+ params.ct_policy_enforcer = &ct_policy_enforcer_;
return new HttpNetworkSession(params);
}
std::unique_ptr<ProxyService> proxy_service_;
scoped_refptr<SSLConfigService> ssl_config_service_;
std::unique_ptr<HttpServerPropertiesImpl> http_server_properties_;
- std::unique_ptr<TransportSecurityState> transport_security_state_;
+ MockCertVerifier cert_verifier_;
+ TransportSecurityState transport_security_state_;
+ MultiLogCTVerifier ct_verifier_;
+ CTPolicyEnforcer ct_policy_enforcer_;
const std::unique_ptr<HttpNetworkSession> session_;
CloseResultWaiter result_waiter_;
MockHttpStream* const mock_stream_; // Owned by |drainer_|.
« no previous file with comments | « net/http/http_network_transaction_ssl_unittest.cc ('k') | net/http/http_stream_factory_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698