QUIC - rename |read_start_time_| to |disk_cache_load_start_time_|.

QUIC - rename |read_start_time_| to |disk_cache_load_start_time_|.

R=wtc@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=256262

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

Patch set 1 LGTM.

1. We may not want to change the histogram's name.

2. If you haven't updated the XML file for histograms, please take care of that
in this CL. Thanks.

https://codereview.chromium.org/192583005/diff/1/net/quic/quic_crypto_client_...
File net/quic/quic_crypto_client_stream.cc (right):

https://codereview.chromium.org/192583005/diff/1/net/quic/quic_crypto_client_...
net/quic/quic_crypto_client_stream.cc:468:
UMA_HISTOGRAM_TIMES("Net.QuicServerInfo.DiskCacheLoadTime",

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

[ramant (doing other things), 2014-03-10 22:01:29]

Hi Wan-Teh,
  The following is the CL for histograms.xml (old habit of separating
histogram.xml changes vs code).

  Renamed the histogram to "...Read..."

https://codereview.chromium.org/193253002/

[ramant (doing other things), 2014-03-10 22:01:49]

The CQ bit was checked by rtenneti@chromium.org

[ramant (doing other things), 2014-03-10 22:02:07]

https://codereview.chromium.org/192583005/diff/1/net/quic/quic_crypto_client_...
File net/quic/quic_crypto_client_stream.cc (right):

https://codereview.chromium.org/192583005/diff/1/net/quic/quic_crypto_client_...
net/quic/quic_crypto_client_stream.cc:468:
UMA_HISTOGRAM_TIMES("Net.QuicServerInfo.DiskCacheLoadTime",
On 2014/03/10 21:54:55, wtc wrote:
> 
> IMPORTANT: we probably should not rename the histogram's name.

Done.

[wtc, 2014-03-10 22:15:45]

Patch set 2 LGTM.

[commit-bot: I haz the power, 2014-03-10 23:10:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/192583005/20001

[commit-bot: I haz the power, 2014-03-11 00:03:27]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/192583005/20001

[commit-bot: I haz the power, 2014-03-11 07:55:53]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-11 07:55:54]

Retried try job too often on win_rel for step(s) unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_rel&nu...

[ramant (doing other things), 2014-03-11 17:06:48]

The CQ bit was checked by rtenneti@chromium.org

[commit-bot: I haz the power, 2014-03-11 17:10:01]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rtenneti@chromium.org/192583005/20001

[commit-bot: I haz the power, 2014-03-11 17:19:50]

Change committed as 256262