Teach gtest how to print QuicConsumedData and QuicEncryptedPacket.

net/quic/quic_protocol.h

Index: net/quic/quic_protocol.h
===================================================================
--- net/quic/quic_protocol.h	(revision 177300)
+++ net/quic/quic_protocol.h	(working copy)
@@ -42,6 +42,9 @@
       : bytes_consumed(bytes_consumed),
         fin_consumed(fin_consumed) {
   }
+  friend std::ostream& operator<<(
+      std::ostream& os, const QuicConsumedData& s);
+
   size_t bytes_consumed;
   bool fin_consumed;

[wtc, 2013/01/17 02:34:58]

There are padding bytes after this bool member.

[Alexander Potapenko, 2013/01/17 08:20:06]

It's better to have this in the code comment rather than in Rietveld.

[wtc, 2013/01/17 18:59:35]

Done.

I verified the bool member was the problem using two methods.
One method was to simply defining the boolean members as
size_t to eliminate padding. That fixed the valgrind memory
errors. (The other method was to use a memset call to
initialize the padding bytes.)

 };
@@ -429,6 +432,9 @@
   QuicEncryptedPacket(char* buffer, size_t length, bool owns_buffer)
       : QuicData(buffer, length, owns_buffer) { }
 
+  friend std::ostream& operator<<(
+      std::ostream& os, const QuicEncryptedPacket& s);
+
   base::StringPiece AssociatedData() const {
     return base::StringPiece(data() + kStartOfHashData, kStartOfEncryptedData);
   }