NTP - Add a histogram tracking the time to load offline content.

NTP - Add a histogram tracking the time to load offline content.

This will be used to determine if a better UI is required while waiting
for disk operations to complete.

BUG=607573
Committed: https://crrev.com/a814ec0b9c16e31e664121ad948ae89070d96229
Cr-Commit-Position: refs/heads/master@{#392358}

[dewittj, 2016-05-06 16:12:16]

dewittj@chromium.org changed reviewers:
+ asvitkine@chromium.org, bauerb@chromium.org

[dewittj, 2016-05-06 16:12:16]

PTAL, thanks

[Alexei Svitkine (slow), 2016-05-06 19:54:17]

lgtm

https://codereview.chromium.org/1949163006/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1949163006/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:33100: +<histogram
name="NewTabPage.OfflineUrlsLoadTime">
Nit: units="ms"

[dewittj, 2016-05-06 19:59:36]

https://codereview.chromium.org/1949163006/diff/1/tools/metrics/histograms/hi...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/1949163006/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:33100: +<histogram
name="NewTabPage.OfflineUrlsLoadTime">
On 2016/05/06 19:54:16, Alexei Svitkine wrote:
> Nit: units="ms"

Done.

[Bernhard Bauer, 2016-05-09 09:51:10]

lgtm

[dewittj, 2016-05-09 16:23:22]

The CQ bit was checked by dewittj@chromium.org

[dewittj, 2016-05-09 16:23:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/1949163006/#ps20001
(title: "add units")

[commit-bot: I haz the power, 2016-05-09 16:23:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1949163006/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1949163006/20001

[commit-bot: I haz the power, 2016-05-09 17:53:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-05-09 17:54:41]

Description was changed from

==========
NTP - Add a histogram tracking the time to load offline content.

This will be used to determine if a better UI is required while waiting
for disk operations to complete.

BUG=607573
==========

to

==========
NTP - Add a histogram tracking the time to load offline content.

This will be used to determine if a better UI is required while waiting
for disk operations to complete.

BUG=607573

Committed: https://crrev.com/a814ec0b9c16e31e664121ad948ae89070d96229
Cr-Commit-Position: refs/heads/master@{#392358}
==========

[commit-bot: I haz the power, 2016-05-09 17:54:42]

Patchset 2 (id:??) landed as
https://crrev.com/a814ec0b9c16e31e664121ad948ae89070d96229
Cr-Commit-Position: refs/heads/master@{#392358}