Minor style cleanup of QuicProtocol

net/quic/quic_protocol.h

Index: net/quic/quic_protocol.h
diff --git a/net/quic/quic_protocol.h b/net/quic/quic_protocol.h
index 902c2659f09cad0115e3db6f5a11b387538c6b62..9f411b5e76d5e4e1fdaaaf0fa47475ad17fe3dad 100644
--- a/net/quic/quic_protocol.h
+++ b/net/quic/quic_protocol.h
@@ -5,6 +5,7 @@
 #ifndef NET_QUIC_QUIC_PROTOCOL_H_
 #define NET_QUIC_QUIC_PROTOCOL_H_
 
+#include <ostream>
 #include <limits>

[jar (doing other things), 2012/10/19 00:26:53]

nit: alphabetize

[Ryan Hamilton, 2012/10/21 22:09:07]

Done.

 #include <utility>
 #include <vector>
@@ -248,30 +249,11 @@ struct NET_EXPORT_PRIVATE QuicAckFragment {
     congestion_info.type = kNone;
   }
 
+  friend std::ostream& operator<<(std::ostream& os, const QuicAckFragment& s);
+
   SentPacketInfo sent_info;
   ReceivedPacketInfo received_info;
   CongestionInfo congestion_info;
-
-  friend std::ostream& operator<<(std::ostream& os, const QuicAckFragment& s) {
-    os << "largest_received: " << s.received_info.largest_received
-       << " time: " << s.received_info.time_received
-       << " missing: ";
-    for (base::hash_set<QuicPacketSequenceNumber>::const_iterator it =
-           s.received_info.missing_packets.begin();
-           it != s.received_info.missing_packets.end(); ++it) {
-      os << *it << " ";
-    }
-
-    os << " least_waiting: " << s.sent_info.least_unacked
-       << " no_retransmit: ";
-    for (base::hash_set<QuicPacketSequenceNumber>::const_iterator it =
-           s.sent_info.non_retransmiting.begin();
-         it != s.sent_info.non_retransmiting.end(); ++it) {
-      os << *it << " ";
-    }
-    os << "\n";
-    return os;
-  }
 };
 
 struct NET_EXPORT_PRIVATE QuicRstStreamFragment {
@@ -329,19 +311,7 @@ struct NET_EXPORT_PRIVATE QuicFecData {
   // The last protected packet's sequence number will be one
   // less than the sequence number of the FEC packet.
   base::StringPiece redundancy;
-  bool operator==(const QuicFecData& other) const {
-    if (fec_group != other.fec_group) {
-      return false;
-    }
-    if (first_protected_packet_sequence_number !=
-        other.first_protected_packet_sequence_number) {
-      return false;
-    }
-    if (redundancy != other.redundancy) {
-      return false;
-    }
-    return true;
-  }
+  bool operator==(const QuicFecData& other) const;
 };
 
 struct NET_EXPORT_PRIVATE QuicPacketData {