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

Unified Diff: components/cronet/url_request_context_config.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: components/cronet/url_request_context_config.cc
diff --git a/components/cronet/url_request_context_config.cc b/components/cronet/url_request_context_config.cc
index 892fd578dc341e083e50130329dcd210bc285f9a..5cc3ea3b89f3cb8f8513db99b71a0446259d418e 100644
--- a/components/cronet/url_request_context_config.cc
+++ b/components/cronet/url_request_context_config.cc
@@ -27,6 +27,8 @@ const char kQuicFieldTrialName[] = "QUIC";
const char kQuicConnectionOptions[] = "connection_options";
const char kQuicStoreServerConfigsInProperties[] =
"store_server_configs_in_properties";
+const char kQuicNumberOfServerConfigsStoredInProperties[] =
+ "number_of_server_configs_stored_in_properties";
const char kQuicDelayTcpRace[] = "delay_tcp_race";
const char kQuicMaxNumberOfLossyConnections[] =
"max_number_of_lossy_connections";
@@ -74,11 +76,21 @@ void ParseAndSetExperimentalOptions(
net::QuicUtils::ParseQuicConnectionOptions(quic_connection_options));
}
+ // TODO(rtenneti): Delete this option after apps stop using it.
+ // Added this for backward compatability.
bool quic_store_server_configs_in_properties = false;
if (quic_args->GetBoolean(kQuicStoreServerConfigsInProperties,
&quic_store_server_configs_in_properties)) {
- context_builder->set_quic_store_server_configs_in_properties(
- quic_store_server_configs_in_properties);
+ context_builder->set_quic_number_of_server_configs_stored_in_properties(
+ net::kMaxQuicServersToPersist);
+ }
+
+ int quic_number_of_server_configs_stored_in_properties = 0;
+ if (quic_args->GetInteger(
+ kQuicNumberOfServerConfigsStoredInProperties,
+ &quic_number_of_server_configs_stored_in_properties)) {
+ context_builder->set_quic_number_of_server_configs_stored_in_properties(
+ quic_number_of_server_configs_stored_in_properties);
}
bool quic_delay_tcp_race = false;

Powered by Google App Engine
This is Rietveld 408576698