QUIC - debugging code to find crash in net::QuicPacket::FecProtectedData.

net/quic/quic_framer.cc

Index: net/quic/quic_framer.cc
diff --git a/net/quic/quic_framer.cc b/net/quic/quic_framer.cc
index ed5c011ff6c011d8762e10f2083cc6177777f59f..66cd8d89b38b99ef2f9fd4e75e3a22e1f295f67d 100644
--- a/net/quic/quic_framer.cc
+++ b/net/quic/quic_framer.cc
@@ -332,7 +332,8 @@ QuicPacket* QuicFramer::BuildDataPacket(const QuicPacketHeader& header,
                                         size_t packet_length) {
   QuicDataWriter writer(packet_length, buffer);
   if (!AppendPacketHeader(header, &writer)) {
-    LOG(DFATAL) << "AppendPacketHeader failed";
+    // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+    CHECK(false) << "AppendPacketHeader failed";
     return nullptr;
   }
 
@@ -343,7 +344,8 @@ QuicPacket* QuicFramer::BuildDataPacket(const QuicPacketHeader& header,
         (header.is_in_fec_group == NOT_IN_FEC_GROUP) &&
         (i == frames.size() - 1);
     if (!AppendTypeByte(frame, no_stream_frame_length, &writer)) {
-      LOG(DFATAL) << "AppendTypeByte failed";
+      // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+      CHECK(false) << "AppendTypeByte failed";
       return nullptr;
     }
 
@@ -354,21 +356,24 @@ QuicPacket* QuicFramer::BuildDataPacket(const QuicPacketHeader& header,
       case STREAM_FRAME:
         if (!AppendStreamFrame(
             *frame.stream_frame, no_stream_frame_length, &writer)) {
-          LOG(DFATAL) << "AppendStreamFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendStreamFrame failed";
           return nullptr;
         }
         break;
       case ACK_FRAME:
         if (!AppendAckFrameAndTypeByte(
                 header, *frame.ack_frame, &writer)) {
-          LOG(DFATAL) << "AppendAckFrameAndTypeByte failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendAckFrameAndTypeByte failed";
           return nullptr;
         }
         break;
       case STOP_WAITING_FRAME:
         if (!AppendStopWaitingFrame(
                 header, *frame.stop_waiting_frame, &writer)) {
-          LOG(DFATAL) << "AppendStopWaitingFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendStopWaitingFrame failed";
           return nullptr;
         }
         break;
@@ -379,38 +384,44 @@ QuicPacket* QuicFramer::BuildDataPacket(const QuicPacketHeader& header,
         break;
       case RST_STREAM_FRAME:
         if (!AppendRstStreamFrame(*frame.rst_stream_frame, &writer)) {
-          LOG(DFATAL) << "AppendRstStreamFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendRstStreamFrame failed";
           return nullptr;
         }
         break;
       case CONNECTION_CLOSE_FRAME:
         if (!AppendConnectionCloseFrame(
                 *frame.connection_close_frame, &writer)) {
-          LOG(DFATAL) << "AppendConnectionCloseFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendConnectionCloseFrame failed";
           return nullptr;
         }
         break;
       case GOAWAY_FRAME:
         if (!AppendGoAwayFrame(*frame.goaway_frame, &writer)) {
-          LOG(DFATAL) << "AppendGoAwayFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendGoAwayFrame failed";
           return nullptr;
         }
         break;
       case WINDOW_UPDATE_FRAME:
         if (!AppendWindowUpdateFrame(*frame.window_update_frame, &writer)) {
-          LOG(DFATAL) << "AppendWindowUpdateFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendWindowUpdateFrame failed";
           return nullptr;
         }
         break;
       case BLOCKED_FRAME:
         if (!AppendBlockedFrame(*frame.blocked_frame, &writer)) {
-          LOG(DFATAL) << "AppendBlockedFrame failed";
+          // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+          CHECK(false) << "AppendBlockedFrame failed";
           return nullptr;
         }
         break;
       default:
         RaiseError(QUIC_INVALID_FRAME_DATA);
-        LOG(DFATAL) << "QUIC_INVALID_FRAME_DATA";
+        // TODO(rtenneti): Revert the CHECKs to LOG(DFATAL).
+        CHECK(false) << "QUIC_INVALID_FRAME_DATA";
         return nullptr;
     }
     ++i;