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

Unified Diff: net/http/http_network_session.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/http/http_network_session.h ('k') | net/http/http_server_properties.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/http/http_network_session.cc
diff --git a/net/http/http_network_session.cc b/net/http/http_network_session.cc
index d3f73aa9dc9176759bfba24ac2518786a6dcfaaf..935453e52352314fe8f0c18d35af0504a56b6700 100644
--- a/net/http/http_network_session.cc
+++ b/net/http/http_network_session.cc
@@ -114,7 +114,7 @@ HttpNetworkSession::Params::Params()
quic_packet_loss_threshold(1.0f),
quic_socket_receive_buffer_size(kQuicSocketReceiveBufferSize),
quic_delay_tcp_race(false),
- quic_store_server_configs_in_properties(false),
+ quic_max_server_configs_stored_in_properties(0u),
quic_clock(NULL),
quic_random(NULL),
quic_max_packet_length(kDefaultMaxPacketSize),
@@ -176,7 +176,7 @@ HttpNetworkSession::HttpNetworkSession(const Params& params)
params.quic_threshold_timeouts_streams_open,
params.quic_socket_receive_buffer_size,
params.quic_delay_tcp_race,
- params.quic_store_server_configs_in_properties,
+ params.quic_max_server_configs_stored_in_properties,
params.quic_close_sessions_on_ip_change,
params.quic_idle_connection_timeout_seconds,
params.quic_migrate_sessions_on_network_change,
@@ -239,6 +239,8 @@ HttpNetworkSession::HttpNetworkSession(const Params& params)
http_server_properties_->SetAlternativeServiceProbabilityThreshold(
params.alternative_service_probability_threshold);
+ http_server_properties_->SetMaxServerConfigsStoredInProperties(
+ params.quic_max_server_configs_stored_in_properties);
}
HttpNetworkSession::~HttpNetworkSession() {
@@ -324,8 +326,8 @@ scoped_ptr<base::Value> HttpNetworkSession::QuicInfoToValue() const {
params_.quic_max_number_of_lossy_connections);
dict->SetDouble("packet_loss_threshold", params_.quic_packet_loss_threshold);
dict->SetBoolean("delay_tcp_race", params_.quic_delay_tcp_race);
- dict->SetBoolean("store_server_configs_in_properties",
- params_.quic_store_server_configs_in_properties);
+ dict->SetInteger("max_server_configs_stored_in_properties",
+ params_.quic_max_server_configs_stored_in_properties);
dict->SetInteger("idle_connection_timeout_seconds",
params_.quic_idle_connection_timeout_seconds);
dict->SetBoolean("disable_preconnect_if_0rtt",
« no previous file with comments | « net/http/http_network_session.h ('k') | net/http/http_server_properties.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698