QUIC - histograms to measure the time spent to read QUIC sever

net/quic/quic_crypto_client_stream.h

Index: net/quic/quic_crypto_client_stream.h
diff --git a/net/quic/quic_crypto_client_stream.h b/net/quic/quic_crypto_client_stream.h
index b191193a2f165d9bc402305f302606bd73ce445b..b4ef3d199602c27cc44a460a6237c225624fb029 100644
--- a/net/quic/quic_crypto_client_stream.h
+++ b/net/quic/quic_crypto_client_stream.h
@@ -8,6 +8,7 @@
 #include <string>
 
 #include "base/memory/weak_ptr.h"
+#include "base/time/time.h"
 #include "net/cert/cert_verify_result.h"
 #include "net/cert/x509_certificate.h"
 #include "net/quic/crypto/proof_verifier.h"
@@ -135,6 +136,10 @@ class NET_EXPORT_PRIVATE QuicCryptoClientStream : public QuicCryptoStream {
   // It must not be used after STATE_LOAD_QUIC_SERVER_INFO_COMPLETE.
   int disk_cache_load_result_;
 
+  // Time when call to WaitForDataReady was made, used for computing time spent
+  // to read QUIC server information from disk cache.
+  base::TimeTicks read_start_time_;

[wtc, 2014/03/10 17:29:28]

Nit: this variable is poorly named because "read" can also mean "reading from
the network".

I suggest renaming it "disk_cache_load_start_time_" (see
"disk_cache_load_result_" above) or "disk_cache_read_start_time_".

[ramant (doing other things), 2014/03/10 20:42:20]

Renamed the variable in https://codereview.chromium.org/192583005/

Thanks Wan-Teh.
Done.

+
   base::WeakPtrFactory<QuicCryptoClientStream> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(QuicCryptoClientStream);