QUIC - rename |read_start_time_| to |disk_cache_load_start_time_|.

net/quic/quic_crypto_client_stream.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/quic/quic_crypto_client_stream.h"
 
 #include "base/metrics/histogram.h"
 #include "net/base/completion_callback.h"
 #include "net/base/net_errors.h"
 #include "net/quic/crypto/crypto_protocol.h"
@@ skipping 405 matching lines @@
 }
 
 int QuicCryptoClientStream::DoLoadQuicServerInfo(
     QuicCryptoClientConfig::CachedState* cached) {
   next_state_ = STATE_SEND_CHLO;
   QuicServerInfo* quic_server_info = cached->quic_server_info();
   if (!quic_server_info) {
     return OK;
   }
 
-  read_start_time_ = base::TimeTicks::Now();
+  disk_cache_load_start_time_ = base::TimeTicks::Now();
   generation_counter_ = cached->generation_counter();
   next_state_ = STATE_LOAD_QUIC_SERVER_INFO_COMPLETE;
 
   // TODO(rtenneti): Use host:port to access QUIC server information from disk
   // cache. If multiple tabs load URLs with same hostname but different
   // ports, all requests except for the first request send InchoateClientHello.
   // Fix the code to handle multiple requests. A possible solution is to wait
   // for the first request to finish and use the data from the disk cache for
   // all requests.
   // We may need to call quic_server_info->Persist later.
@@ skipping 21 matching lines @@
     QuicCryptoClientConfig::CachedState* cached) {
   next_state_ = STATE_SEND_CHLO;
 
   // If someone else already saved a server config, we don't want to overwrite
   // it. Also, if someone else saved a server config and then cleared it (so
   // cached->IsEmpty() is true), we still want to load from QuicServerInfo.
   if (!cached->IsEmpty()) {
     return;
   }
 
-  UMA_HISTOGRAM_TIMES("Net.QuicServerInfo.DiskCacheReadTime",
-                      base::TimeTicks::Now() - read_start_time_);
+  UMA_HISTOGRAM_TIMES("Net.QuicServerInfo.DiskCacheLoadTime",

[wtc, 2014/03/10 21:54:55]

IMPORTANT: we probably should not rename the histogram's name.

[ramant (doing other things), 2014/03/10 22:02:08]

Done.

+                      base::TimeTicks::Now() - disk_cache_load_start_time_);
 
   if (disk_cache_load_result_ != OK || !cached->LoadQuicServerInfo(
           session()->connection()->clock()->WallNow())) {
     // It is ok to proceed to STATE_SEND_CHLO when we cannot load QuicServerInfo
     // from the disk cache.
     DCHECK(cached->IsEmpty());
     DVLOG(1) << "Empty server_config";
     return;
   }
 
   ProofVerifier* verifier = crypto_config_->proof_verifier();
   if (!verifier) {
     // If no verifier is set then we don't check the certificates.
     cached->SetProofValid();
   } else if (!cached->signature().empty()) {
     next_state_ = STATE_VERIFY_PROOF;
   }
 }
 
 }  // namespace net