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

Unified Diff: net/quic/quic_framer_test.cc

Issue 1472573002: These packets got reformatted at some point, maybe before the clang-format off annotation was added. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@107711408
Patch Set: Created 5 years, 1 month 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 | « no previous file | net/quic/quic_write_blocked_list.h » ('j') | net/tools/quic/quic_server_bin.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/quic_framer_test.cc
diff --git a/net/quic/quic_framer_test.cc b/net/quic/quic_framer_test.cc
index 5e4ad4af68e5025ac002cbd8a71cc427e4eef614..547d907e216252f11dc7a355602b760b46146393 100644
--- a/net/quic/quic_framer_test.cc
+++ b/net/quic/quic_framer_test.cc
@@ -3110,58 +3110,22 @@ TEST_P(QuicFramerTest, BuildStreamFramePacketWithVersionFlag) {
// public flags (version, 8 byte connection_id)
0x3D,
// connection_id
- 0x10,
- 0x32,
- 0x54,
- 0x76,
- 0x98,
- 0xBA,
- 0xDC,
- 0xFE,
+ 0x10, 0x32, 0x54, 0x76, 0x98, 0xBA, 0xDC, 0xFE,
// version tag
- 'Q',
- '0',
- GetQuicVersionDigitTens(),
- GetQuicVersionDigitOnes(),
+ 'Q', '0', GetQuicVersionDigitTens(), GetQuicVersionDigitOnes(),
// packet number
- 0xBC,
- 0x9A,
- 0x78,
- 0x56,
- 0x34,
- 0x12,
+ 0xBC, 0x9A, 0x78, 0x56, 0x34, 0x12,
// private flags (entropy)
0x01,
// frame type (stream frame with fin and no length)
0xDF,
// stream id
- 0x04,
- 0x03,
- 0x02,
- 0x01,
+ 0x04, 0x03, 0x02, 0x01,
// offset
- 0x54,
- 0x76,
- 0x10,
- 0x32,
- 0xDC,
- 0xFE,
- 0x98,
- 0xBA,
+ 0x54, 0x76, 0x10, 0x32, 0xDC, 0xFE, 0x98, 0xBA,
// data
- 'h',
- 'e',
- 'l',
- 'l',
- 'o',
- ' ',
- 'w',
- 'o',
- 'r',
- 'l',
- 'd',
- '!',
+ 'h', 'e', 'l', 'l', 'o', ' ', 'w', 'o', 'r', 'l', 'd', '!',
};
// clang-format on
@@ -3180,19 +3144,9 @@ TEST_P(QuicFramerTest, BuildVersionNegotiationPacket) {
// public flags (version, 8 byte connection_id)
0x0D,
// connection_id
- 0x10,
- 0x32,
- 0x54,
- 0x76,
- 0x98,
- 0xBA,
- 0xDC,
- 0xFE,
+ 0x10, 0x32, 0x54, 0x76, 0x98, 0xBA, 0xDC, 0xFE,
// version tag
- 'Q',
- '0',
- GetQuicVersionDigitTens(),
- GetQuicVersionDigitOnes(),
+ 'Q', '0', GetQuicVersionDigitTens(), GetQuicVersionDigitOnes(),
};
// clang-format on
« no previous file with comments | « no previous file | net/quic/quic_write_blocked_list.h » ('j') | net/tools/quic/quic_server_bin.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698