Fix the logic for computing the number of truncated QUIC ACK frames sent.

net/quic/quic_connection_logger.cc

Index: net/quic/quic_connection_logger.cc
diff --git a/net/quic/quic_connection_logger.cc b/net/quic/quic_connection_logger.cc
index 8157848fff62ff06eba358da203057cf82aa70b8..be7d8389d53e4f907db163e9a723f4be4dd3f0e5 100644
--- a/net/quic/quic_connection_logger.cc
+++ b/net/quic/quic_connection_logger.cc
@@ -371,7 +371,15 @@ void QuicConnectionLogger::OnFrameAddedToPacket(const QuicFrame& frame) {
       net_log_.AddEvent(
           NetLog::TYPE_QUIC_SESSION_ACK_FRAME_SENT,
           base::Bind(&NetLogQuicAckFrameCallback, frame.ack_frame));
-      if (frame.ack_frame->is_truncated)
+      size_t num_ranges = 0;
+      QuicPacketSequenceNumber last_missing = 0;
+      for (size_t i = 0; i < frame.ack_frame->missing_packets.size(); ++i) {
+        if (frame.ack_frame->missing_packets[i] != last_missing + 1) {

[Ian Swett, 2014/09/08 17:27:58]

It's annoying to have to perform this logic in multiple spots, but LGTM.

+          ++num_ranges;
+        }
+        last_missing = frame.ack_frame->missing_packets[i];
+      }
+      if (num_ranges >= numeric_limits<uint8>::max())
         ++num_truncated_acks_sent_;
       break;
     case CONGESTION_FEEDBACK_FRAME: