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

Unified Diff: net/tools/quic/quic_simple_server_stream_test.cc

Issue 1783783003: Add a QuicCompressedCertsCache instance to QuicDispatcher, plumbing to QuicServerSessionBase but no… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@116273065
Patch Set: 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
« no previous file with comments | « net/tools/quic/quic_simple_server_session_test.cc ('k') | net/tools/quic/test_tools/quic_dispatcher_peer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_simple_server_stream_test.cc
diff --git a/net/tools/quic/quic_simple_server_stream_test.cc b/net/tools/quic/quic_simple_server_stream_test.cc
index 32775745692ed3395f60f3be0902e9b4a3f094ee..594e34ca7f4cff00ae90bcdaa0fad342b356ff14 100644
--- a/net/tools/quic/quic_simple_server_stream_test.cc
+++ b/net/tools/quic/quic_simple_server_stream_test.cc
@@ -84,13 +84,16 @@ class MockQuicSimpleServerSession : public QuicSimpleServerSession {
public:
const size_t kMaxStreamsForTest = 100;
- explicit MockQuicSimpleServerSession(QuicConnection* connection,
- MockQuicServerSessionVisitor* owner,
- QuicCryptoServerConfig* crypto_config)
- : QuicSimpleServerSession(::net::test::DefaultQuicConfig(),
+ explicit MockQuicSimpleServerSession(
+ QuicConnection* connection,
+ MockQuicServerSessionVisitor* owner,
+ QuicCryptoServerConfig* crypto_config,
+ QuicCompressedCertsCache* compressed_certs_cache)
+ : QuicSimpleServerSession(DefaultQuicConfig(),
connection,
owner,
- crypto_config) {
+ crypto_config,
+ compressed_certs_cache) {
set_max_open_incoming_streams(kMaxStreamsForTest);
set_max_open_outgoing_streams(kMaxStreamsForTest);
ON_CALL(*this, WritevData(_, _, _, _, _))
@@ -152,7 +155,12 @@ class QuicSimpleServerStreamTest
QuicCryptoServerConfig::TESTING,
QuicRandom::GetInstance(),
::net::test::CryptoTestUtils::ProofSourceForTesting())),
- session_(connection_, session_owner_, crypto_config_.get()),
+ compressed_certs_cache_(
+ QuicCompressedCertsCache::kQuicCompressedCertsCacheSize),
+ session_(connection_,
+ session_owner_,
+ crypto_config_.get(),
+ &compressed_certs_cache_),
body_("hello world") {
FLAGS_quic_always_log_bugs_for_tests = true;
SpdyHeaderBlock request_headers;
@@ -197,6 +205,7 @@ class QuicSimpleServerStreamTest
StrictMock<MockConnection>* connection_;
StrictMock<MockQuicServerSessionVisitor>* session_owner_;
std::unique_ptr<QuicCryptoServerConfig> crypto_config_;
+ QuicCompressedCertsCache compressed_certs_cache_;
StrictMock<MockQuicSimpleServerSession> session_;
QuicSimpleServerStreamPeer* stream_; // Owned by session_.
string headers_string_;
« no previous file with comments | « net/tools/quic/quic_simple_server_session_test.cc ('k') | net/tools/quic/test_tools/quic_dispatcher_peer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698