relnote: Moving many QUIC DFATALS over to QUIC_BUG

net/quic/quic_unacked_packet_map.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_unacked_packet_map.h"
 
 #include "base/logging.h"
-#include "base/stl_util.h"
+#include "net/quic/quic_bug_tracker.h"
 #include "net/quic/quic_connection_stats.h"
 #include "net/quic/quic_flags.h"
 #include "net/quic/quic_utils_chromium.h"
 
 using std::max;
 
 namespace net {
 
 QuicUnackedPacketMap::QuicUnackedPacketMap()
     : largest_sent_packet_(0),
@@ skipping 67 matching lines @@
   }
 }
 
 void QuicUnackedPacketMap::TransferRetransmissionInfo(
     QuicPacketNumber old_packet_number,
     QuicPacketNumber new_packet_number,
     TransmissionType transmission_type,
     TransmissionInfo* info) {
   if (old_packet_number < least_unacked_ ||
       old_packet_number > largest_sent_packet_) {
-    LOG(DFATAL) << "Old TransmissionInfo no longer exists for:"
-                << old_packet_number << " least_unacked:" << least_unacked_
-                << " largest_sent:" << largest_sent_packet_;
+    QUIC_BUG << "Old TransmissionInfo no longer exists for:"
+             << old_packet_number << " least_unacked:" << least_unacked_
+             << " largest_sent:" << largest_sent_packet_;
     return;
   }
   DCHECK_GE(new_packet_number, least_unacked_ + unacked_packets_.size());
   DCHECK_NE(NOT_RETRANSMISSION, transmission_type);
 
   TransmissionInfo* transmission_info =
       &unacked_packets_.at(old_packet_number - least_unacked_);
   RetransmittableFrames* frames = transmission_info->retransmittable_frames;
   transmission_info->retransmittable_frames = nullptr;
   for (AckListenerWrapper& wrapper : transmission_info->ack_listeners) {
@@ skipping 243 matching lines @@
 QuicTime QuicUnackedPacketMap::GetLastPacketSentTime() const {
   UnackedPacketMap::const_reverse_iterator it = unacked_packets_.rbegin();
   while (it != unacked_packets_.rend()) {
     if (it->in_flight) {
       LOG_IF(DFATAL, it->sent_time == QuicTime::Zero())
           << "Sent time can never be zero for a packet in flight.";
       return it->sent_time;
     }
     ++it;
   }
-  LOG(DFATAL) << "GetLastPacketSentTime requires in flight packets.";
+  QUIC_BUG << "GetLastPacketSentTime requires in flight packets.";
   return QuicTime::Zero();
 }
 
 size_t QuicUnackedPacketMap::GetNumUnackedPacketsDebugOnly() const {
   size_t unacked_packet_count = 0;
   QuicPacketNumber packet_number = least_unacked_;
   for (UnackedPacketMap::const_iterator it = unacked_packets_.begin();
        it != unacked_packets_.end(); ++it, ++packet_number) {
     if (!IsPacketUseless(packet_number, *it)) {
       ++unacked_packet_count;
@@ skipping 31 matching lines @@
     }
   }
   return false;
 }
 
 QuicPacketNumber QuicUnackedPacketMap::GetLeastUnacked() const {
   return least_unacked_;
 }
 
 }  // namespace net