Add histograms and NetLog for host cache persistence

Add histograms and NetLog for host cache persistence

BUG=605149
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2958023002
Cr-Commit-Position: refs/heads/master@{#483526}
Committed: https://chromium.googlesource.com/chromium/src/+/00eb7a2dfd3224a58de38a4a202b976e7f0b118f

[mgersh, 2017-06-27 14:16:30]

Description was changed from

==========
Add histograms and NetLog for host cache persistence

BUG=605149
==========

to

==========
Add histograms and NetLog for host cache persistence

BUG=605149
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

[mgersh, 2017-06-27 14:19:38]

mgersh@chromium.org changed reviewers:
+ pauljensen@chromium.org

[mgersh, 2017-06-27 14:19:39]

I changed my mind and did this part as a separate CL instead.

[pauljensen, 2017-06-28 14:12:01]

lgtm modulo one comment

https://codereview.chromium.org/2958023002/diff/1/components/cronet/android/c...
File components/cronet/android/cronet_url_request_context_adapter.cc (right):

https://codereview.chromium.org/2958023002/diff/1/components/cronet/android/c...
components/cronet/android/cronet_url_request_context_adapter.cc:670:
prefs_init_end - prefs_init_start);
{
    SCOPED_UMA_HISTOGRAM_TIMER("Net.Cronet.PrefsInitTime");
    pref_service_ = factory.Create(registry.get());
}

[mgersh, 2017-06-28 14:21:07]

mgersh@chromium.org changed reviewers:
+ holte@chromium.org

[mgersh, 2017-06-28 14:21:08]

holte: PTAL at histograms

https://codereview.chromium.org/2958023002/diff/1/components/cronet/android/c...
File components/cronet/android/cronet_url_request_context_adapter.cc (right):

https://codereview.chromium.org/2958023002/diff/1/components/cronet/android/c...
components/cronet/android/cronet_url_request_context_adapter.cc:670:
prefs_init_end - prefs_init_start);
On 2017/06/28 14:12:01, pauljensen wrote:
> {
>     SCOPED_UMA_HISTOGRAM_TIMER("Net.Cronet.PrefsInitTime");
>     pref_service_ = factory.Create(registry.get());
> }

Done.

[Steven Holte, 2017-06-29 18:35:23]

lgtm

[mgersh, 2017-06-29 18:37:11]

The CQ bit was checked by mgersh@chromium.org

[mgersh, 2017-06-29 18:37:12]

The patchset sent to the CQ was uploaded after l-g-t-m from
pauljensen@chromium.org
Link to the patchset: https://codereview.chromium.org/2958023002/#ps20001
(title: "use scoped timer")

[commit-bot: I haz the power, 2017-06-29 18:37:35]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-29 20:02:53]

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

[commit-bot: I haz the power, 2017-06-29 20:02:54]

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[mgersh, 2017-06-29 21:07:56]

The CQ bit was checked by mgersh@chromium.org

[commit-bot: I haz the power, 2017-06-29 21:08:30]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-29 22:06:43]

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

[commit-bot: I haz the power, 2017-06-29 22:06:46]

Try jobs failed on following builders:
  android_cronet_tester on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)

[mgersh, 2017-06-29 22:07:38]

On 2017/06/29 22:06:46, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   android_cronet_tester on master.tryserver.chromium.android (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.chromium.android/builders/android_cron...)

Cronet bots are broken :(
crbug.com/738183

[mgersh, 2017-06-29 22:20:39]

The CQ bit was checked by mgersh@chromium.org

[commit-bot: I haz the power, 2017-06-29 22:20:55]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-29 22:47:55]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1498774839559710,
"parent_rev": "97e4ec20d584f59fe5a701cde8b29257b0a0b31d", "commit_rev":
"00eb7a2dfd3224a58de38a4a202b976e7f0b118f"}

[commit-bot: I haz the power, 2017-06-29 22:48:09]

Description was changed from

==========
Add histograms and NetLog for host cache persistence

BUG=605149
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester
==========

to

==========
Add histograms and NetLog for host cache persistence

BUG=605149
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

Review-Url: https://codereview.chromium.org/2958023002
Cr-Commit-Position: refs/heads/master@{#483526}
Committed:
https://chromium.googlesource.com/chromium/src/+/00eb7a2dfd3224a58de38a4a202b...
==========

[commit-bot: I haz the power, 2017-06-29 22:48:10]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/00eb7a2dfd3224a58de38a4a202b...