Fix a QUIC bug where previously undecryptable packets were not decrypted

net/quic/quic_packet_creator.cc

Index: net/quic/quic_packet_creator.cc
diff --git a/net/quic/quic_packet_creator.cc b/net/quic/quic_packet_creator.cc
index 2ac874742d9732b2afd5210e91b3d99e2ab9f756..48147f70d64c5bfc336a37de1e58c3b7502faa83 100644
--- a/net/quic/quic_packet_creator.cc
+++ b/net/quic/quic_packet_creator.cc
@@ -421,6 +421,16 @@ bool QuicPacketCreator::ShouldRetransmit(const QuicFrame& frame) {
 
 bool QuicPacketCreator::AddFrame(const QuicFrame& frame,
                                  bool save_retransmittable_frames) {
+#ifndef NDEBUG
+  if (frame.type == ACK_FRAME) {

[Ryan Hamilton, 2014/01/29 00:52:52]

nit:I think I would use a switch statement here.  You might want to consider
adding an operator<< method to QuicFrame which actually does this.  Then you
could remove the #ifdef:

DVLOG(1) << "Adding: " << frame;

[ramant (doing other things), 2014/01/29 02:45:38]

Done.

+    DVLOG(1) << "AddFrame ACK_FRAME: " << *(frame.ack_frame);
+  } else if (frame.type == RST_STREAM_FRAME) {
+    DVLOG(1) << "AddFrame RST_STREAM_FRAME: " << *(frame.rst_stream_frame);
+  } else if (frame.type == STREAM_FRAME) {
+    DVLOG(1) << "AddFrame STREAM_FRAME: " << *(frame.stream_frame);
+  }
+#endif
+
   size_t frame_len = framer_->GetSerializedFrameLength(
       frame, BytesFree(), queued_frames_.empty(), true,
       options()->send_sequence_number_length);