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

Unified Diff: components/cronet/url_request_context_config_unittest.cc

Issue 2318053004: Remove obsolete QUIC disabling code. (Closed)
Patch Set: Rebase Created 4 years, 3 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_unittest.cc
diff --git a/components/cronet/url_request_context_config_unittest.cc b/components/cronet/url_request_context_config_unittest.cc
index 0c82e96b21a97ec3e57cf47ba7dbe5b956196a06..7d0a54cac1a7814784940aad7638cc1b6be5de1f 100644
--- a/components/cronet/url_request_context_config_unittest.cc
+++ b/components/cronet/url_request_context_config_unittest.cc
@@ -40,10 +40,8 @@ TEST(URLRequestContextConfigTest, TestExperimentalOptionPassing) {
// JSON encoded experimental options.
"{\"QUIC\":{\"max_server_configs_stored_in_properties\":2,"
"\"delay_tcp_race\":true,"
- "\"max_number_of_lossy_connections\":10,"
"\"prefer_aes\":true,"
"\"user_agent_id\":\"Custom QUIC UAID\","
- "\"packet_loss_threshold\":0.5,"
"\"idle_connection_timeout_seconds\":300,"
"\"close_sessions_on_ip_change\":true,"
"\"race_cert_verification\":true,"
@@ -97,10 +95,6 @@ TEST(URLRequestContextConfigTest, TestExperimentalOptionPassing) {
// Check prefer_aes.
EXPECT_TRUE(params->quic_prefer_aes);
- // Check max_number_of_lossy_connections and packet_loss_threshold.
- EXPECT_EQ(10, params->quic_max_number_of_lossy_connections);
- EXPECT_FLOAT_EQ(0.5f, params->quic_packet_loss_threshold);
-
// Check idle_connection_timeout_seconds.
EXPECT_EQ(300, params->quic_idle_connection_timeout_seconds);

Powered by Google App Engine
This is Rietveld 408576698