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

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

Issue 2359503003: Landing Recent QUIC changes until 9:58 AM, Sep 17, 2016 UTC (Closed)
Patch Set: Reverted flag flip. 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
« no previous file with comments | « net/quic/core/quic_framer.cc ('k') | net/quic/core/quic_headers_stream.h » ('j') | 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 1d00d732a308b154a7d882ed2361f67e06d4e4f2..0ce097352de99b25e06355b2decc2a74e1d64b00 100644
--- a/net/quic/core/quic_framer_test.cc
+++ b/net/quic/core/quic_framer_test.cc
@@ -4904,8 +4904,7 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithVersionFlag) {
unsigned char packet[] = {
// public flags (version, 8 byte connection_id)
static_cast<unsigned char>(
- (FLAGS_quic_remove_v33_hacks &&
- framer_.version() > QUIC_VERSION_32) ? 0x39 : 0x3D),
+ framer_.version() > QUIC_VERSION_32 ? 0x39 : 0x3D),
// connection_id
0x10, 0x32, 0x54, 0x76, 0x98, 0xBA, 0xDC, 0xFE,
// version tag
@@ -4926,8 +4925,7 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithVersionFlag) {
};
unsigned char packet_34[] = {
// public flags (version, 8 byte connection_id)
- static_cast<unsigned char>(
- FLAGS_quic_remove_v33_hacks ? 0x39 : 0x3D),
+ 0x39,
// connection_id
0x10, 0x32, 0x54, 0x76, 0x98, 0xBA, 0xDC, 0xFE,
// version tag
@@ -5059,8 +5057,7 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithBothVersionAndMultipathFlag) {
unsigned char packet[] = {
// public flags (8 byte connection_id)
static_cast<unsigned char>(
- (FLAGS_quic_remove_v33_hacks &&
- framer_.version() > QUIC_VERSION_32) ? 0x79 : 0x7D),
+ framer_.version() > QUIC_VERSION_32 ? 0x79 : 0x7D),
// connection_id
0x10, 0x32, 0x54, 0x76,
0x98, 0xBA, 0xDC, 0xFE,
@@ -5088,8 +5085,7 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithBothVersionAndMultipathFlag) {
};
unsigned char packet_34[] = {
// public flags (8 byte connection_id)
- static_cast<unsigned char>(
- FLAGS_quic_remove_v33_hacks ? 0x79 : 0x7D),
+ 0x79,
// connection_id
0x10, 0x32, 0x54, 0x76,
0x98, 0xBA, 0xDC, 0xFE,
« no previous file with comments | « net/quic/core/quic_framer.cc ('k') | net/quic/core/quic_headers_stream.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698