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

Unified Diff: chrome/browser/io_thread_unittest.cc

Issue 811073004: QUIC - don't load data from disk cache if alternate protocol map doesn't (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: renamed variable Created 6 years 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 | « chrome/browser/io_thread.cc ('k') | net/http/http_network_session.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/io_thread_unittest.cc
diff --git a/chrome/browser/io_thread_unittest.cc b/chrome/browser/io_thread_unittest.cc
index e11676a6f81367be8dc6ddf5a0e37417a5c5c2af..7d34219237000cc0f8f9ccca800d55881ef3eb5d 100644
--- a/chrome/browser/io_thread_unittest.cc
+++ b/chrome/browser/io_thread_unittest.cc
@@ -133,6 +133,7 @@ TEST_F(IOThreadTest, EnableQuicFromFieldTrialGroup) {
EXPECT_FALSE(params.quic_always_require_handshake_confirmation);
EXPECT_FALSE(params.quic_disable_connection_pooling);
EXPECT_EQ(0, params.quic_load_server_info_timeout_ms);
+ EXPECT_FALSE(params.quic_disable_loading_server_info_for_new_servers);
}
TEST_F(IOThreadTest, EnableQuicFromCommandLine) {
@@ -295,6 +296,15 @@ TEST_F(IOThreadTest, QuicLoadServerInfoTimeoutFromFieldTrialParams) {
EXPECT_EQ(50, params.quic_load_server_info_timeout_ms);
}
+TEST_F(IOThreadTest, QuicDisableLoadingServerInfoForNewServers) {
+ field_trial_group_ = "Enabled";
+ field_trial_params_["disable_loading_server_info_for_new_servers"] = "true";
+ ConfigureQuicGlobals();
+ net::HttpNetworkSession::Params params;
+ InitializeNetworkSessionParams(&params);
+ EXPECT_TRUE(params.quic_disable_loading_server_info_for_new_servers);
Ryan Hamilton 2014/12/18 21:12:55 please add an EXPECT_FALSE for this field up to th
ramant (doing other things) 2014/12/18 23:05:55 Done.
+}
+
TEST_F(IOThreadTest,
AlternateProtocolProbabilityThresholdFromFlag) {
command_line_.AppendSwitchASCII("alternate-protocol-probability-threshold",
« no previous file with comments | « chrome/browser/io_thread.cc ('k') | net/http/http_network_session.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698