Add histograms for tracking the number of client hellos send. This will let us understand how frequ…

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 493dac848d5f12dfc4a6bd3275a5b97d6a0f6fa7..a5b3121f86dfd91c08b26df07c91b3eb8a960b7f 100644
--- a/net/quic/quic_client_session.cc
+++ b/net/quic/quic_client_session.cc
@@ -6,6 +6,7 @@
 
 #include "base/callback_helpers.h"
 #include "base/message_loop.h"
+#include "base/metrics/histogram.h"
 #include "base/stl_util.h"
 #include "base/string_number_conversions.h"
 #include "base/values.h"
@@ -53,6 +54,16 @@ QuicClientSession::~QuicClientSession() {
   DCHECK(callback_.is_null());
   connection()->set_debug_visitor(NULL);
   net_log_.EndEvent(NetLog::TYPE_QUIC_SESSION);
+  UMA_HISTOGRAM_BOOLEAN("Net.QuicCryptoHandshakeConfirmed",
+                        IsCryptoHandshakeConfirmed());
+  UMA_HISTOGRAM_BOOLEAN("Net.QuicEncryptionEstablished",
+                        IsEncryptionEstablished());
+  UMA_HISTOGRAM_COUNTS("Net.QuicNumSentClientHellos",

[jar (doing other things), 2013/05/30 21:08:33]

maybe better:

UMA_HISTOGRAM_ENUMERATION("...", sample, max+1);

+                       crypto_stream_->num_sent_client_hellos());
+  if (IsCryptoHandshakeConfirmed()) {
+    UMA_HISTOGRAM_COUNTS("Net.QuicCryptoHandshakeConfirmedNumSentClientHellos",

[jar (doing other things), 2013/05/30 21:08:33]

nit: To get better ordering of histograms... alphabetize near the similar one:

Net.QuicNumSentClientHellosCryptoHandshakeConfirmed

+                         crypto_stream_->num_sent_client_hellos());
+  }
 }
 
 QuicReliableClientStream* QuicClientSession::CreateOutgoingReliableStream() {