Test divisor for zero before generating statistic

net/quic/quic_client_session.cc

Index: net/quic/quic_client_session.cc
diff --git a/net/quic/quic_client_session.cc b/net/quic/quic_client_session.cc
index 0cbc92cc82cbabf8b26637e9a1f70a2a80f22317..352f7f568423772ab570d3711021b6cf00995c75 100644
--- a/net/quic/quic_client_session.cc
+++ b/net/quic/quic_client_session.cc
@@ -244,9 +244,9 @@ QuicClientSession::~QuicClientSession() {
     }
   }
   const QuicConnectionStats stats = connection()->GetStats();
-  uint64 reordering =
-      GG_UINT64_C(100) * stats.max_time_reordering_us / stats.min_rtt_us;
-    UMA_HISTOGRAM_CUSTOM_COUNTS("Net.QuicSession.MaxReorderingTime",
+  uint64 reordering = (stats.min_rtt_us == 0) ? GG_UINT64_C(100)

[Ryan Hamilton, 2014/05/24 21:34:21]

nit: no need for GG_UINT64_C() around the 100 here. I needed it below to ensure
that the multiplication was 64 bit, so it didn't overflow.

+      : GG_UINT64_C(100) * stats.max_time_reordering_us / stats.min_rtt_us;
+  UMA_HISTOGRAM_CUSTOM_COUNTS("Net.QuicSession.MaxReorderingTime",
                                 reordering, 0, 100, 50);
   if (stats.min_rtt_us > 100) {
     UMA_HISTOGRAM_CUSTOM_COUNTS("Net.QuicSession.MaxReorderingTimeLongRtt",