Log foreground duration for failed provisional loads.

Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

This metric will also help to measure the impact of the delayed navigation
experiment, where we expect there to be an increased number of
failed provisional loads.

BUG=705058
Review-Url: https://codereview.chromium.org/2771293002
Cr-Commit-Position: refs/heads/master@{#459576}
Committed: https://chromium.googlesource.com/chromium/src/+/8db499251a795b754dbef05b1d73a5e877001da8

[Bryan McQuade, 2017-03-24 20:14:32]

The CQ bit was checked by bmcquade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-24 20:15:16]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Bryan McQuade, 2017-03-24 20:16:00]

Description was changed from

==========
Log foreground duration for failed provisional loads.

BUG=
==========

to

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

BUG=705058
==========

[Bryan McQuade, 2017-03-24 20:17:31]

bmcquade@chromium.org changed reviewers:
+ csharrison@chromium.org

[Bryan McQuade, 2017-03-24 20:18:00]

Description was changed from

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

BUG=705058
==========

to

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

This metric will also help to measure the impact of the delayed navigation
experiment, where we expect there to be an increased number of
provisional loads.

BUG=705058
==========

[Bryan McQuade, 2017-03-24 20:18:06]

PTAL, thanks!

[Bryan McQuade, 2017-03-24 20:20:44]

Description was changed from

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

This metric will also help to measure the impact of the delayed navigation
experiment, where we expect there to be an increased number of
provisional loads.

BUG=705058
==========

to

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

This metric will also help to measure the impact of the delayed navigation
experiment, where we expect there to be an increased number of
failed provisional loads.

BUG=705058
==========

[Bryan McQuade, 2017-03-24 20:23:47]

The CQ bit was checked by bmcquade@chromium.org to run a CQ dry run

[Bryan McQuade, 2017-03-24 20:24:19]

bmcquade@chromium.org changed reviewers:
+ isherman@chromium.org

[commit-bot: I haz the power, 2017-03-24 20:24:22]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Bryan McQuade, 2017-03-24 20:24:31]

isherman, PTAL for histograms.xml, thanks!

[Bryan McQuade, 2017-03-24 20:32:22]

The CQ bit was checked by bmcquade@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-03-24 20:33:39]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Charlie Harrison, 2017-03-24 20:44:10]

LGTM

[commit-bot: I haz the power, 2017-03-24 21:46:45]

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

[commit-bot: I haz the power, 2017-03-24 21:46:46]

Dry run: This issue passed the CQ dry run.

[Ilya Sherman, 2017-03-24 22:08:26]

histograms.xml lgtm

[Bryan McQuade, 2017-03-24 22:17:50]

The CQ bit was checked by bmcquade@chromium.org

[commit-bot: I haz the power, 2017-03-24 22:18:52]

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-03-24 22:26:15]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1490393870974200,
"parent_rev": "b455655aabeb57f21c947ed8cdaf4761d5d20fd8", "commit_rev":
"8db499251a795b754dbef05b1d73a5e877001da8"}

[commit-bot: I haz the power, 2017-03-24 22:27:04]

Description was changed from

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

This metric will also help to measure the impact of the delayed navigation
experiment, where we expect there to be an increased number of
failed provisional loads.

BUG=705058
==========

to

==========
Log foreground duration for failed provisional loads.

UKM logs foreground duration for both committed and provisional loads.
This change makes core UMA metrics more consistent with UKM.

This metric will also help to measure the impact of the delayed navigation
experiment, where we expect there to be an increased number of
failed provisional loads.

BUG=705058

Review-Url: https://codereview.chromium.org/2771293002
Cr-Commit-Position: refs/heads/master@{#459576}
Committed:
https://chromium.googlesource.com/chromium/src/+/8db499251a795b754dbef05b1d73...
==========

[commit-bot: I haz the power, 2017-03-24 22:27:06]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/8db499251a795b754dbef05b1d73...