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

Unified Diff: net/quic/quic_protocol_test.cc

Issue 266243004: Clang format slam. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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
Index: net/quic/quic_protocol_test.cc
diff --git a/net/quic/quic_protocol_test.cc b/net/quic/quic_protocol_test.cc
index b1eae45d25117baeb8a1efe0d4937045dd352f79..63fc938a55d4b593ec92e11a9894515ec9385072 100644
--- a/net/quic/quic_protocol_test.cc
+++ b/net/quic/quic_protocol_test.cc
@@ -16,12 +16,13 @@ TEST(QuicProtocolTest, AdjustErrorForVersion) {
<< "Any additions to QuicRstStreamErrorCode require an addition to "
<< "AdjustErrorForVersion and this associated test.";
- EXPECT_EQ(QUIC_STREAM_NO_ERROR,
- AdjustErrorForVersion(QUIC_RST_FLOW_CONTROL_ACCOUNTING,
- QUIC_VERSION_17));
- EXPECT_EQ(QUIC_RST_FLOW_CONTROL_ACCOUNTING, AdjustErrorForVersion(
+ EXPECT_EQ(
+ QUIC_STREAM_NO_ERROR,
+ AdjustErrorForVersion(QUIC_RST_FLOW_CONTROL_ACCOUNTING, QUIC_VERSION_17));
+ EXPECT_EQ(
QUIC_RST_FLOW_CONTROL_ACCOUNTING,
- static_cast<QuicVersion>(QUIC_VERSION_17 + 1)));
+ AdjustErrorForVersion(QUIC_RST_FLOW_CONTROL_ACCOUNTING,
+ static_cast<QuicVersion>(QUIC_VERSION_17 + 1)));
}
TEST(QuicProtocolTest, MakeQuicTag) {
@@ -51,16 +52,17 @@ TEST(QuicProtocolTest, InsertMissingPacketsBetween) {
QuicPacketSequenceNumber i = 4;
for (SequenceNumberSet::iterator it = received_info.missing_packets.begin();
- it != received_info.missing_packets.end(); ++it, ++i) {
+ it != received_info.missing_packets.end();
+ ++it, ++i) {
EXPECT_EQ(i, *it);
}
}
TEST(QuicProtocolTest, QuicVersionToQuicTag) {
- // If you add a new version to the QuicVersion enum you will need to add a new
- // case to QuicVersionToQuicTag, otherwise this test will fail.
+// If you add a new version to the QuicVersion enum you will need to add a new
+// case to QuicVersionToQuicTag, otherwise this test will fail.
- // TODO(rtenneti): Enable checking of Log(ERROR) messages.
+// TODO(rtenneti): Enable checking of Log(ERROR) messages.
#if 0
// Any logs would indicate an unsupported version which we don't expect.
ScopedMockLog log(kDoNotCaptureLogsYet);
@@ -82,7 +84,7 @@ TEST(QuicProtocolTest, QuicVersionToQuicTag) {
}
TEST(QuicProtocolTest, QuicVersionToQuicTagUnsupported) {
- // TODO(rtenneti): Enable checking of Log(ERROR) messages.
+// TODO(rtenneti): Enable checking of Log(ERROR) messages.
#if 0
// TODO(rjshade): Change to DFATAL once we actually support multiple versions,
// and QuicConnectionTest::SendVersionNegotiationPacket can be changed to use
@@ -96,10 +98,10 @@ TEST(QuicProtocolTest, QuicVersionToQuicTagUnsupported) {
}
TEST(QuicProtocolTest, QuicTagToQuicVersion) {
- // If you add a new version to the QuicVersion enum you will need to add a new
- // case to QuicTagToQuicVersion, otherwise this test will fail.
+// If you add a new version to the QuicVersion enum you will need to add a new
+// case to QuicTagToQuicVersion, otherwise this test will fail.
- // TODO(rtenneti): Enable checking of Log(ERROR) messages.
+// TODO(rtenneti): Enable checking of Log(ERROR) messages.
#if 0
// Any logs would indicate an unsupported version which we don't expect.
ScopedMockLog log(kDoNotCaptureLogsYet);
@@ -126,7 +128,7 @@ TEST(QuicProtocolTest, QuicTagToQuicVersion) {
}
TEST(QuicProtocolTest, QuicTagToQuicVersionUnsupported) {
- // TODO(rtenneti): Enable checking of Log(ERROR) messages.
+// TODO(rtenneti): Enable checking of Log(ERROR) messages.
#if 0
ScopedMockLog log(kDoNotCaptureLogsYet);
#ifndef NDEBUG

Powered by Google App Engine
This is Rietveld 408576698