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

Unified Diff: net/tools/quic/quic_dispatcher_test.cc

Issue 2875333002: Landing Recent QUIC changes until Mon May 8 21:42:46 2017 +0000 (Closed)
Patch Set: Created 3 years, 7 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/tools/quic/quic_dispatcher.cc ('k') | net/tools/quic/quic_simple_server_stream.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/tools/quic/quic_dispatcher_test.cc
diff --git a/net/tools/quic/quic_dispatcher_test.cc b/net/tools/quic/quic_dispatcher_test.cc
index f14425d093853176e4bcf6a2105af09c8ac8d543..6e9b8ec741701a7f31f9e342ad583647a1303e04 100644
--- a/net/tools/quic/quic_dispatcher_test.cc
+++ b/net/tools/quic/quic_dispatcher_test.cc
@@ -529,10 +529,11 @@ TEST_F(QuicDispatcherTest, TooBigSeqNoPacketToTimeWaitListManager) {
}
TEST_F(QuicDispatcherTest, SupportedVersionsChangeInFlight) {
- static_assert(arraysize(kSupportedQuicVersions) == 5u,
+ static_assert(arraysize(kSupportedQuicVersions) == 6u,
"Supported versions out of sync");
FLAGS_quic_reloadable_flag_quic_enable_version_38 = true;
FLAGS_quic_reloadable_flag_quic_enable_version_39 = true;
+ SetQuicFlag(&FLAGS_quic_enable_version_40, true);
QuicSocketAddress client_address(QuicIpAddress::Loopback4(), 1);
server_address_ = QuicSocketAddress(QuicIpAddress::Any4(), 5);
QuicConnectionId connection_id = 1;
@@ -570,6 +571,31 @@ TEST_F(QuicDispatcherTest, SupportedVersionsChangeInFlight) {
ProcessPacket(client_address, connection_id, true, QuicVersionMax(),
SerializeCHLO(), PACKET_8BYTE_CONNECTION_ID,
PACKET_6BYTE_PACKET_NUMBER, 1);
+ // Turn off version 40.
+ SetQuicFlag(&FLAGS_quic_enable_version_40, false);
+ ++connection_id;
+ EXPECT_CALL(*dispatcher_, CreateQuicSession(connection_id, client_address))
+ .Times(0);
+ ProcessPacket(client_address, connection_id, true, QUIC_VERSION_40,
+ SerializeCHLO(), PACKET_8BYTE_CONNECTION_ID,
+ PACKET_6BYTE_PACKET_NUMBER, 1);
+
+ // Turn on version 40.
+ SetQuicFlag(&FLAGS_quic_enable_version_40, true);
+ ++connection_id;
+ EXPECT_CALL(*dispatcher_, CreateQuicSession(connection_id, client_address))
+ .WillOnce(testing::Return(CreateSession(
+ dispatcher_.get(), config_, connection_id, client_address,
+ &mock_helper_, &mock_alarm_factory_, &crypto_config_,
+ QuicDispatcherPeer::GetCache(dispatcher_.get()), &session1_)));
+ EXPECT_CALL(*reinterpret_cast<MockQuicConnection*>(session1_->connection()),
+ ProcessUdpPacket(_, _, _))
+ .WillOnce(testing::WithArgs<2>(
+ Invoke(CreateFunctor(&QuicDispatcherTest::ValidatePacket,
+ base::Unretained(this), connection_id))));
+ ProcessPacket(client_address, connection_id, true, QUIC_VERSION_40,
+ SerializeCHLO(), PACKET_8BYTE_CONNECTION_ID,
+ PACKET_6BYTE_PACKET_NUMBER, 1);
// Turn off version 39.
FLAGS_quic_reloadable_flag_quic_enable_version_39 = false;
« no previous file with comments | « net/tools/quic/quic_dispatcher.cc ('k') | net/tools/quic/quic_simple_server_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698