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

Unified Diff: net/quic/quic_config_test.cc

Issue 559373003: Landing Recent QUIC Changes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix compiler errors Created 6 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
« no previous file with comments | « net/quic/quic_config.cc ('k') | net/quic/quic_connection.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_config_test.cc
diff --git a/net/quic/quic_config_test.cc b/net/quic/quic_config_test.cc
index 6cbd4ff177d3b2019bf635d0c01c89f9d5e40ed0..a1f93efcf8a4d78cb7fdfd652acdbcf866b4ada9 100644
--- a/net/quic/quic_config_test.cc
+++ b/net/quic/quic_config_test.cc
@@ -83,8 +83,8 @@ TEST_F(QuicConfigTest, ProcessClientHello) {
cgst.push_back(kQBIC);
client_config.set_congestion_feedback(cgst, kQBIC);
client_config.set_idle_connection_state_lifetime(
- QuicTime::Delta::FromSeconds(2 * kDefaultTimeoutSecs),
- QuicTime::Delta::FromSeconds(kDefaultTimeoutSecs));
+ QuicTime::Delta::FromSeconds(2 * kMaximumIdleTimeoutSecs),
+ QuicTime::Delta::FromSeconds(kMaximumIdleTimeoutSecs));
client_config.set_max_streams_per_connection(
2 * kDefaultMaxStreamsPerConnection, kDefaultMaxStreamsPerConnection);
client_config.SetInitialRoundTripTimeUsToSend(
@@ -108,7 +108,7 @@ TEST_F(QuicConfigTest, ProcessClientHello) {
EXPECT_EQ(QUIC_NO_ERROR, error);
EXPECT_TRUE(config_.negotiated());
EXPECT_EQ(kQBIC, config_.congestion_feedback());
- EXPECT_EQ(QuicTime::Delta::FromSeconds(kDefaultTimeoutSecs),
+ EXPECT_EQ(QuicTime::Delta::FromSeconds(kMaximumIdleTimeoutSecs),
config_.idle_connection_state_lifetime());
EXPECT_EQ(kDefaultMaxStreamsPerConnection,
config_.max_streams_per_connection());
@@ -136,8 +136,8 @@ TEST_F(QuicConfigTest, ProcessServerHello) {
cgst.push_back(kQBIC);
server_config.set_congestion_feedback(cgst, kQBIC);
server_config.set_idle_connection_state_lifetime(
- QuicTime::Delta::FromSeconds(kDefaultTimeoutSecs / 2),
- QuicTime::Delta::FromSeconds(kDefaultTimeoutSecs / 2));
+ QuicTime::Delta::FromSeconds(kMaximumIdleTimeoutSecs / 2),
+ QuicTime::Delta::FromSeconds(kMaximumIdleTimeoutSecs / 2));
server_config.set_max_streams_per_connection(
kDefaultMaxStreamsPerConnection / 2,
kDefaultMaxStreamsPerConnection / 2);
@@ -159,7 +159,7 @@ TEST_F(QuicConfigTest, ProcessServerHello) {
EXPECT_EQ(QUIC_NO_ERROR, error);
EXPECT_TRUE(config_.negotiated());
EXPECT_EQ(kQBIC, config_.congestion_feedback());
- EXPECT_EQ(QuicTime::Delta::FromSeconds(kDefaultTimeoutSecs / 2),
+ EXPECT_EQ(QuicTime::Delta::FromSeconds(kMaximumIdleTimeoutSecs / 2),
config_.idle_connection_state_lifetime());
EXPECT_EQ(kDefaultMaxStreamsPerConnection / 2,
config_.max_streams_per_connection());
@@ -240,8 +240,8 @@ TEST_F(QuicConfigTest, MissingValueInSHLO) {
TEST_F(QuicConfigTest, OutOfBoundSHLO) {
QuicConfig server_config;
server_config.set_idle_connection_state_lifetime(
- QuicTime::Delta::FromSeconds(2 * kDefaultTimeoutSecs),
- QuicTime::Delta::FromSeconds(2 * kDefaultTimeoutSecs));
+ QuicTime::Delta::FromSeconds(2 * kMaximumIdleTimeoutSecs),
+ QuicTime::Delta::FromSeconds(2 * kMaximumIdleTimeoutSecs));
CryptoHandshakeMessage msg;
server_config.ToHandshakeMessage(&msg);
« no previous file with comments | « net/quic/quic_config.cc ('k') | net/quic/quic_connection.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698