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

Unified Diff: net/quic/quic_stream_factory.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: Fix comments in Patch set 4 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
« no previous file with comments | « net/quic/quic_stream_factory.h ('k') | net/quic/quic_stream_factory_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_stream_factory.cc
diff --git a/net/quic/quic_stream_factory.cc b/net/quic/quic_stream_factory.cc
index 1eea730924a5be8982dc2d3ed664ea82c6d9cc67..a0d45824c4fd3ef29b104c55a242e7c41d0a7e63 100644
--- a/net/quic/quic_stream_factory.cc
+++ b/net/quic/quic_stream_factory.cc
@@ -586,7 +586,7 @@ QuicStreamFactory::QuicStreamFactory(
int threshold_timeouts_with_open_streams,
int socket_receive_buffer_size,
bool delay_tcp_race,
- bool store_server_configs_in_properties,
+ int max_server_configs_stored_in_properties,
bool close_sessions_on_ip_change,
int idle_connection_timeout_seconds,
bool migrate_sessions_on_network_change,
@@ -635,7 +635,6 @@ QuicStreamFactory::QuicStreamFactory(
yield_after_packets_(kQuicYieldAfterPacketsRead),
yield_after_duration_(QuicTime::Delta::FromMilliseconds(
kQuicYieldAfterDurationMilliseconds)),
- store_server_configs_in_properties_(store_server_configs_in_properties),
close_sessions_on_ip_change_(close_sessions_on_ip_change),
migrate_sessions_on_network_change_(
migrate_sessions_on_network_change &&
@@ -673,7 +672,7 @@ QuicStreamFactory::QuicStreamFactory(
// When disk cache is used to store the server configs, HttpCache code calls
// |set_quic_server_info_factory| if |quic_server_info_factory_| wasn't
// created.
- if (store_server_configs_in_properties_) {
+ if (max_server_configs_stored_in_properties > 0) {
quic_server_info_factory_.reset(
new PropertiesBasedQuicServerInfoFactory(http_server_properties_));
}
@@ -1566,7 +1565,7 @@ void QuicStreamFactory::MaybeInitialize() {
}
}
- if (!store_server_configs_in_properties_)
+ if (http_server_properties_->max_server_configs_stored_in_properties() == 0)
return;
// Create a temporary QuicServerInfo object to deserialize and to populate the
// in-memory crypto server config cache.
« no previous file with comments | « net/quic/quic_stream_factory.h ('k') | net/quic/quic_stream_factory_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698