Add UMA metrics for Bluetooth system daemon

Add UMA metrics for Bluetooth system daemon

This adds the following histograms and corresponding enums definitions.

[Histograms]
- BlueZ.ReasonOfDisconnection
- BlueZ.ResultOfAdvertisementRegistration
- BlueZ.ResultOfConnection
- BlueZ.ResultOfPairing
- BlueZ.TimeLengthOfAdvertisement
- BlueZ.TimeLengthOfDiscoverable
- BlueZ.TimeLengthOfDiscovering
- BlueZ.TimeLengthOfPairing
- BlueZ.TimeLengthOfSetupConnection
- BlueZ.TypeOfDiscovery
- BlueZ.TypeOfFoundDevice
[Enums]
- BlueZReasonOfDisconnection
- BlueZResultOfAdvertisementRegistration
- BlueZResultOfConnection
- BlueZResultOfPairing
- BlueZTypeOfDiscovery
- BlueZTypeOfFoundDevice

BUG=chromium:724720
Review-Url: https://codereview.chromium.org/2956143002
Cr-Commit-Position: refs/heads/master@{#483175}
Committed: https://chromium.googlesource.com/chromium/src/+/6c4da1390d590da133fa547a590edfbbcb292df4

[Miao, 2017-06-27 22:32:13]

mcchou@chromium.org changed reviewers:
+ asvitkine@chromium.org, josephsih@chromium.org, mcchou@chromium.org,
rkc@chromium.org

[Miao, 2017-06-27 22:32:13]

Please see https://chromium-review.googlesource.com/c/538910/ for metrics
reference.

[Miao, 2017-06-27 23:15:42]

mcchou@chromium.org changed reviewers:
+ sonnysasaka@chromium.org

[Alexei Svitkine (slow), 2017-06-28 16:58:43]

lgtm

https://codereview.chromium.org/2956143002/diff/20001/tools/metrics/histogram...
File tools/metrics/histograms/histograms.xml (right):

https://codereview.chromium.org/2956143002/diff/20001/tools/metrics/histogram...
tools/metrics/histograms/histograms.xml:6519: +    of disconnection request.
Mention that these are specific to ChromeOS in all of their descriptions.

[mcchou, 2017-06-28 17:20:35]

The CQ bit was checked by mcchou@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-06-28 17:20:53]

Dry run: CQ is trying da patch.

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

[Miao, 2017-06-28 17:22:10]

On 2017/06/28 16:58:43, Alexei Svitkine (slow) wrote:
> lgtm
> 
>
https://codereview.chromium.org/2956143002/diff/20001/tools/metrics/histogram...
> File tools/metrics/histograms/histograms.xml (right):
> 
>
https://codereview.chromium.org/2956143002/diff/20001/tools/metrics/histogram...
> tools/metrics/histograms/histograms.xml:6519: +    of disconnection request.
> Mention that these are specific to ChromeOS in all of their descriptions.
Done.

[commit-bot: I haz the power, 2017-06-28 18:57:55]

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

[commit-bot: I haz the power, 2017-06-28 18:57:56]

Dry run: This issue passed the CQ dry run.

[Miao, 2017-06-28 22:25:45]

The CQ bit was checked by mcchou@chromium.org

[Miao, 2017-06-28 22:25:45]

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/2956143002/#ps40001
(title: "Add "Chrome OS specific" to the descriptions of histograms.")

[commit-bot: I haz the power, 2017-06-28 22:26:16]

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-28 22:30:16]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498688745324350,
"parent_rev": "ac130a51d426be4e44cf2d8743e284cbd360e408", "commit_rev":
"6c4da1390d590da133fa547a590edfbbcb292df4"}

[commit-bot: I haz the power, 2017-06-28 22:30:30]

Description was changed from

==========
Add UMA metrics for Bluetooth system daemon

This adds the following histograms and corresponding enums definitions.

[Histograms]
- BlueZ.ReasonOfDisconnection
- BlueZ.ResultOfAdvertisementRegistration
- BlueZ.ResultOfConnection
- BlueZ.ResultOfPairing
- BlueZ.TimeLengthOfAdvertisement
- BlueZ.TimeLengthOfDiscoverable
- BlueZ.TimeLengthOfDiscovering
- BlueZ.TimeLengthOfPairing
- BlueZ.TimeLengthOfSetupConnection
- BlueZ.TypeOfDiscovery
- BlueZ.TypeOfFoundDevice
[Enums]
- BlueZReasonOfDisconnection
- BlueZResultOfAdvertisementRegistration
- BlueZResultOfConnection
- BlueZResultOfPairing
- BlueZTypeOfDiscovery
- BlueZTypeOfFoundDevice

BUG=chromium:724720
==========

to

==========
Add UMA metrics for Bluetooth system daemon

This adds the following histograms and corresponding enums definitions.

[Histograms]
- BlueZ.ReasonOfDisconnection
- BlueZ.ResultOfAdvertisementRegistration
- BlueZ.ResultOfConnection
- BlueZ.ResultOfPairing
- BlueZ.TimeLengthOfAdvertisement
- BlueZ.TimeLengthOfDiscoverable
- BlueZ.TimeLengthOfDiscovering
- BlueZ.TimeLengthOfPairing
- BlueZ.TimeLengthOfSetupConnection
- BlueZ.TypeOfDiscovery
- BlueZ.TypeOfFoundDevice
[Enums]
- BlueZReasonOfDisconnection
- BlueZResultOfAdvertisementRegistration
- BlueZResultOfConnection
- BlueZResultOfPairing
- BlueZTypeOfDiscovery
- BlueZTypeOfFoundDevice

BUG=chromium:724720

Review-Url: https://codereview.chromium.org/2956143002
Cr-Commit-Position: refs/heads/master@{#483175}
Committed:
https://chromium.googlesource.com/chromium/src/+/6c4da1390d590da133fa547a590e...
==========

[commit-bot: I haz the power, 2017-06-28 22:30:32]

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

[Takashi Toyoshima, 2017-06-29 18:31:31]

QQ: did you see any presubmit warnings or errors on the histograms.xml?