Add a histogram to track the number of QUIC packets received by a QUIC

net/quic/quic_stream_factory.cc

Index: net/quic/quic_stream_factory.cc
diff --git a/net/quic/quic_stream_factory.cc b/net/quic/quic_stream_factory.cc
index 63c5a7165232ceaeacfccdc9da09eba89adc068c..96381b617244b7bf8f3cb274901fc30c10458601 100644
--- a/net/quic/quic_stream_factory.cc
+++ b/net/quic/quic_stream_factory.cc
@@ -425,6 +425,7 @@ void QuicStreamFactory::OnIdleSession(QuicClientSession* session) {
 }
 
 void QuicStreamFactory::OnSessionGoingAway(QuicClientSession* session) {
+  const QuicConnectionStats& stats = session->connection()->GetStats();
   const AliasSet& aliases = session_aliases_[session];
   for (AliasSet::const_iterator it = aliases.begin(); it != aliases.end();
        ++it) {
@@ -440,6 +441,8 @@ void QuicStreamFactory::OnSessionGoingAway(QuicClientSession* session) {
       // differently so that we still race TCP but we don't consider the
       // session connected until the handshake has been confirmed.
       http_server_properties_->SetBrokenAlternateProtocol(it->first);
+      UMA_HISTOGRAM_COUNTS("Net.QuicHandshakeNotConfirmedNumPacketsReceived",
+                           stats.packets_received);
     } else {
       QuicConnectionStats stats = session->connection()->GetStats();

[jar (doing other things), 2014/03/06 22:28:25]

nit: Can we delete this since you pulled it out of the loop to common code?

[Ryan Hamilton, 2014/03/06 22:42:58]

Done.  Also added a continue to the previous block, and removed the else.

       HttpServerProperties::NetworkStats network_stats;