[NTP Snippets] Add UMA for database load time

[NTP Snippets] Add UMA for database load time

BUG=645070
Committed: https://crrev.com/4a711318f9b8f69695797de99147c6dbefe6062b
Cr-Commit-Position: refs/heads/master@{#426163}

[Marc Treib, 2016-10-18 14:55:42]

treib@chromium.org changed reviewers:
+ mvanouwerkerk@chromium.org

[Marc Treib, 2016-10-18 14:55:43]

PTAL!

[Michael van Ouwerkerk, 2016-10-18 14:58:19]

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

[commit-bot: I haz the power, 2016-10-18 14:58:32]

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

[Michael van Ouwerkerk, 2016-10-18 15:04:32]

lgtm

[Marc Treib, 2016-10-18 15:07:48]

treib@chromium.org changed reviewers:
+ rkaplow@chromium.org

[Marc Treib, 2016-10-18 15:07:48]

+rkaplow for histograms.xml, PTAL!

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

https://codereview.chromium.org/2423193003/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:37802: +<histogram
name="NewTabPage.Snippets.DatabaseLoadTime">
Is there a unit I can/should use for UMA_HISTOGRAM_MEDIUM_TIMES? ms?

[commit-bot: I haz the power, 2016-10-18 16:02:06]

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

[commit-bot: I haz the power, 2016-10-18 16:02:06]

Dry run: This issue passed the CQ dry run.

[rkaplow, 2016-10-18 20:24:05]

lgtm

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

https://codereview.chromium.org/2423193003/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:37802: +<histogram
name="NewTabPage.Snippets.DatabaseLoadTime">
On 2016/10/18 15:07:48, Marc Treib wrote:
> Is there a unit I can/should use for UMA_HISTOGRAM_MEDIUM_TIMES? ms?

yes, ms

[Marc Treib, 2016-10-19 08:21:50]

The CQ bit was checked by treib@chromium.org

[Marc Treib, 2016-10-19 08:21:50]

The patchset sent to the CQ was uploaded after l-g-t-m from
rkaplow@chromium.org, mvanouwerkerk@chromium.org
Link to the patchset: https://codereview.chromium.org/2423193003/#ps20001
(title: "ms")

[commit-bot: I haz the power, 2016-10-19 09:30: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, 2016-10-19 11:13:11]

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

[commit-bot: I haz the power, 2016-10-19 11:13:12]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Marc Treib, 2016-10-19 11:26:57]

The CQ bit was checked by treib@chromium.org

[commit-bot: I haz the power, 2016-10-19 11:27:26]

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, 2016-10-19 12:48:54]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:07:28]

Description was changed from

==========
[NTP Snippets] Add UMA for database load time

BUG=645070
==========

to

==========
[NTP Snippets] Add UMA for database load time

BUG=645070

Committed: https://crrev.com/4a711318f9b8f69695797de99147c6dbefe6062b
Cr-Commit-Position: refs/heads/master@{#426163}
==========

[commit-bot: I haz the power, 2016-10-21 13:07:29]

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