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

Unified Diff: net/quic/core/quic_framer_test.cc

Issue 2426823003: Re-enable the QUIC v33 hacks. (Closed)
Patch Set: Rebase to tip of tree Created 4 years, 2 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/core/quic_framer.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/core/quic_framer_test.cc
diff --git a/net/quic/core/quic_framer_test.cc b/net/quic/core/quic_framer_test.cc
index cd43369c91ee00888cce0c74065d1c423d3799ae..1ae77ee27c4a3aa712f7aa2ed7ac116286d41e76 100644
--- a/net/quic/core/quic_framer_test.cc
+++ b/net/quic/core/quic_framer_test.cc
@@ -4232,7 +4232,8 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithVersionFlag) {
unsigned char packet[] = {
// public flags (version, 8 byte connection_id)
static_cast<unsigned char>(
- framer_.version() > QUIC_VERSION_32 ? 0x39 : 0x3D),
+ (FLAGS_quic_remove_v33_hacks2 &&
+ framer_.version() > QUIC_VERSION_32) ? 0x39 : 0x3D),
// connection_id
0x10, 0x32, 0x54, 0x76, 0x98, 0xBA, 0xDC, 0xFE,
// version tag
@@ -4253,7 +4254,8 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithVersionFlag) {
};
unsigned char packet_34[] = {
// public flags (version, 8 byte connection_id)
- 0x39,
+ static_cast<unsigned char>(
+ FLAGS_quic_remove_v33_hacks2 ? 0x39 : 0x3D),
// connection_id
0x10, 0x32, 0x54, 0x76, 0x98, 0xBA, 0xDC, 0xFE,
// version tag
@@ -4383,7 +4385,8 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithBothVersionAndMultipathFlag) {
unsigned char packet[] = {
// public flags (8 byte connection_id)
static_cast<unsigned char>(
- framer_.version() > QUIC_VERSION_32 ? 0x79 : 0x7D),
+ (FLAGS_quic_remove_v33_hacks2 &&
+ framer_.version() > QUIC_VERSION_32) ? 0x79 : 0x7D),
// connection_id
0x10, 0x32, 0x54, 0x76,
0x98, 0xBA, 0xDC, 0xFE,
@@ -4411,7 +4414,8 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithBothVersionAndMultipathFlag) {
};
unsigned char packet_34[] = {
// public flags (8 byte connection_id)
- 0x79,
+ static_cast<unsigned char>(
+ FLAGS_quic_remove_v33_hacks2 ? 0x79 : 0x7D),
// connection_id
0x10, 0x32, 0x54, 0x76,
0x98, 0xBA, 0xDC, 0xFE,
« no previous file with comments | « net/quic/core/quic_framer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698