| 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
|
|
|