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

Unified Diff: net/quic/quic_stream_factory_test.cc

Issue 1572753003: QUIC - Allow cronet apps to specify how many server configs are to be (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: load all server configs from disk Created 4 years, 11 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/quic/quic_stream_factory_test.cc
diff --git a/net/quic/quic_stream_factory_test.cc b/net/quic/quic_stream_factory_test.cc
index 477bb5252f3cc652edf91e619115159930b15ebc..ff9bdd6b99c074fb2e8b7ae7adcab9035f6eb696 100644
--- a/net/quic/quic_stream_factory_test.cc
+++ b/net/quic/quic_stream_factory_test.cc
@@ -149,7 +149,7 @@ class QuicStreamFactoryTest : public ::testing::TestWithParam<TestParams> {
threshold_public_resets_post_handshake_(2),
receive_buffer_size_(0),
delay_tcp_race_(false),
- store_server_configs_in_properties_(false),
+ number_of_server_configs_stored_in_properties_(0),
close_sessions_on_ip_change_(false),
idle_connection_timeout_seconds_(kIdleConnectionTimeoutSeconds) {
clock_->AdvanceTime(QuicTime::Delta::FromSeconds(1));
@@ -170,7 +170,7 @@ class QuicStreamFactoryTest : public ::testing::TestWithParam<TestParams> {
max_number_of_lossy_connections_, packet_loss_threshold_,
max_disabled_reasons_, threshold_timeouts_with_open_streams_,
threshold_public_resets_post_handshake_, receive_buffer_size_,
- delay_tcp_race_, store_server_configs_in_properties_,
+ delay_tcp_race_, number_of_server_configs_stored_in_properties_,
close_sessions_on_ip_change_, idle_connection_timeout_seconds_,
QuicTagVector()));
factory_->set_require_confirmation(false);
@@ -302,7 +302,7 @@ class QuicStreamFactoryTest : public ::testing::TestWithParam<TestParams> {
int threshold_public_resets_post_handshake_;
int receive_buffer_size_;
bool delay_tcp_race_;
- bool store_server_configs_in_properties_;
+ int number_of_server_configs_stored_in_properties_;
bool close_sessions_on_ip_change_;
int idle_connection_timeout_seconds_;
};
@@ -2496,7 +2496,7 @@ TEST_P(QuicStreamFactoryTest, EnableDelayTcpRace) {
}
TEST_P(QuicStreamFactoryTest, MaybeInitialize) {
- store_server_configs_in_properties_ = true;
+ number_of_server_configs_stored_in_properties_ = kMaxQuicServersToPersist;
idle_connection_timeout_seconds_ = 500;
Initialize();
ProofVerifyDetailsChromium verify_details = DefaultProofVerifyDetails();

Powered by Google App Engine
This is Rietveld 408576698