QUIC - Minor change to keep the code similar to internal source tree.

+ Minor fixes to using order, comments and chromium indentation nits.
+ Use EXPECT_EQ instead of EXPECT_TRUE.

Merge internal change: 69401047

R=rch@chromium.org,wtc@chromium.org,

Review URL: https://codereview.chromium.org/345453003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278319 0039d316-1c4b-4281-b951-d872f2087c98

net/quic/congestion_control/send_algorithm_simulator.h

@@ -23,10 +23,10 @@ class SendAlgorithmSimulator {
   struct SentPacket {
     SentPacket(QuicPacketSequenceNumber sequence_number,
                QuicTime send_time,
-               QuicTime ack_time) :
-        sequence_number(sequence_number),
-        send_time(send_time),
-        ack_time(ack_time) {}
+               QuicTime ack_time)
+        : sequence_number(sequence_number),
+          send_time(send_time),
+          ack_time(ack_time) {}
     QuicPacketSequenceNumber sequence_number;
     QuicTime send_time;
     QuicTime ack_time;

net/quic/crypto/crypto_server_test.cc

@@ -439,7 +439,7 @@ TEST(CryptoServerConfigGenerationTest, SCIDIsHashOfServerConfig) {
   hash->Finish(digest, sizeof(digest));
 
   ASSERT_EQ(scid.size(), sizeof(digest));
-  EXPECT_TRUE(0 == memcmp(digest, scid_str.data(), sizeof(digest)));
+  EXPECT_EQ(0, memcmp(digest, scid_str.data(), sizeof(digest)));
 }
 
 class CryptoServerTestNoConfig : public CryptoServerTest {

net/quic/quic_protocol.h

@@ -161,7 +161,7 @@ enum FecProtection {
   MAY_FEC_PROTECT    // Callee does not have to but may FEC protect this data.
 };
 
-// Indicates FEC policy
+// Indicates FEC policy.
 enum FecPolicy {
   FEC_PROTECT_ALWAYS,   // All data in the stream should be FEC protected.
   FEC_PROTECT_OPTIONAL  // Data in the stream does not need FEC protection.

net/quic/test_tools/quic_test_utils.cc

@@ -23,8 +23,8 @@ using base::StringPiece;
 using std::max;
 using std::min;
 using std::string;
-using testing::_;
 using testing::AnyNumber;
+using testing::_;
 
 namespace net {
 namespace test {

net/tools/quic/test_tools/packet_dropping_test_writer.cc

@@ -35,8 +35,7 @@ class WriteUnblockedAlarm : public QuicAlarm::Delegate {
 // later point.
 class DelayAlarm : public QuicAlarm::Delegate {
  public:
-  explicit DelayAlarm(PacketDroppingTestWriter* writer)
-      : writer_(writer) {}
+  explicit DelayAlarm(PacketDroppingTestWriter* writer) : writer_(writer) {}
 
   virtual QuicTime OnAlarm() OVERRIDE {
     return writer_->ReleaseOldPackets();

net/tools/quic/test_tools/quic_test_utils.cc

@@ -10,10 +10,10 @@
 #include "net/tools/quic/quic_epoll_connection_helper.h"
 
 using base::StringPiece;
-using net::test::kInitialFlowControlWindowForTest;
 using net::test::MakeAckFrame;
 using net::test::MockHelper;
 using net::test::QuicConnectionPeer;
+using net::test::kInitialFlowControlWindowForTest;
 
 namespace net {
 namespace tools {