Add histogram definitions for Chrome OS boot mode.

Add histogram definitions for Chrome OS boot mode.

This adds enum type definitions and histogram declarations for boot
mode metrics recorded by the Chrome OS boot process. See also
https://chromium-review.googlesource.com/#/c/400739

BUG=chromium:658207
TEST=None

Committed: https://crrev.com/64815089e2fbe13e92661d8c81d7c6c05c6b172e
Cr-Commit-Position: refs/heads/master@{#428670}

[Mattias Nissler (ping if slow), 2016-10-27 10:57:10]

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

[Mattias Nissler (ping if slow), 2016-10-27 12:00:07]

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

[commit-bot: I haz the power, 2016-10-27 12:00:27]

Dry run: 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-27 12:54:04]

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

[commit-bot: I haz the power, 2016-10-27 12:54:05]

Dry run: This issue passed the CQ dry run.

[Ilya Sherman, 2016-10-28 02:24:07]

LGTM

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

https://codereview.chromium.org/2445993005/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:43062: +</histogram>
nit: Maybe use a histogram_suffixes element to reduce the repetition from these
three entries?

[Mattias Nissler (ping if slow), 2016-10-28 11:53:09]

Switched to use suffixes. Still looks good?

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

https://codereview.chromium.org/2445993005/diff/1/tools/metrics/histograms/hi...
tools/metrics/histograms/histograms.xml:43062: +</histogram>
On 2016/10/28 02:24:07, Ilya Sherman wrote:
> nit: Maybe use a histogram_suffixes element to reduce the repetition from
these
> three entries?

Done.

[Ilya Sherman, 2016-10-28 18:30:53]

LGTM

[Ilya Sherman, 2016-10-28 18:30:59]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2016-10-28 18:31:39]

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-28 20:34:02]

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

[commit-bot: I haz the power, 2016-10-28 20:34:03]

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

[Mattias Nissler (ping if slow), 2016-10-31 08:56:23]

The CQ bit was checked by mnissler@chromium.org

[commit-bot: I haz the power, 2016-10-31 08:56: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, 2016-10-31 09:49:55]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-31 09:51:46]

Description was changed from

==========
Add histogram definitions for Chrome OS boot mode.

This adds enum type definitions and histogram declarations for boot
mode metrics recorded by the Chrome OS boot process. See also
https://chromium-review.googlesource.com/#/c/400739

BUG=chromium:658207
TEST=None
==========

to

==========
Add histogram definitions for Chrome OS boot mode.

This adds enum type definitions and histogram declarations for boot
mode metrics recorded by the Chrome OS boot process. See also
https://chromium-review.googlesource.com/#/c/400739

BUG=chromium:658207
TEST=None

Committed: https://crrev.com/64815089e2fbe13e92661d8c81d7c6c05c6b172e
Cr-Commit-Position: refs/heads/master@{#428670}
==========

[commit-bot: I haz the power, 2016-10-31 09:51:47]

Patchset 3 (id:??) landed as
https://crrev.com/64815089e2fbe13e92661d8c81d7c6c05c6b172e
Cr-Commit-Position: refs/heads/master@{#428670}