android: Record the source of account change notifications from GSA.

android: Record the source of account change notifications from GSA.

GSA can notify Chrome of an account change through either a service
call, or through a broadcast. This histogram records the number of
account changes of each type that Chrome sees.

This is intended to act as a migration tracker and sanity check as we
migrate from the service to the broadcast mechanism. The total count
should stay approximately the same, while the proportion of each type
changes.

BUG=614388
Review-Url: https://codereview.chromium.org/2638653005
Cr-Commit-Position: refs/heads/master@{#444705}
Committed: https://chromium.googlesource.com/chromium/src/+/f8e7b55cbaf9147033f46e730a029347d3ae5baa

[Benoit L, 2017-01-16 14:35:18]

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

[commit-bot: I haz the power, 2017-01-16 14:35:30]

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

[Benoit L, 2017-01-16 14:54:49]

lizeb@chromium.org changed reviewers:
+ yusufo@chromium.org

[commit-bot: I haz the power, 2017-01-16 15:52:03]

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

[commit-bot: I haz the power, 2017-01-16 15:52:03]

Dry run: This issue passed the CQ dry run.

[Yusuf, 2017-01-17 07:45:42]

lgtm

[Benoit L, 2017-01-17 10:36:24]

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

[Benoit L, 2017-01-17 10:36:25]

+isherman for the historgram part.
PTAL, thanks :-)

[Ilya Sherman, 2017-01-18 02:15:52]

Metrics LGTM

[Benoit L, 2017-01-18 09:40:11]

The CQ bit was checked by lizeb@chromium.org

[commit-bot: I haz the power, 2017-01-18 09:40:44]

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-01-18 09:47:01]

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

[commit-bot: I haz the power, 2017-01-18 09:47: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...)

[Benoit L, 2017-01-18 09:56:14]

lizeb@chromium.org changed reviewers:
+ tedchoc@chromium.org

[Benoit L, 2017-01-18 09:56:15]

+tedchoc for owners approval.
Thanks!

[Ted C, 2017-01-18 18:23:15]

On 2017/01/18 09:56:15, Benoit L wrote:
> +tedchoc for owners approval.
> Thanks!

lgtm -- thoughts on adding an OWNERS file to:
chrome/android/java/src/org/chromium/chrome/browser/gsa/ ?

If you want to do that as part of this change, then that works
for me as well.

[Benoit L, 2017-01-19 09:44:55]

The CQ bit was checked by lizeb@chromium.org

[Benoit L, 2017-01-19 09:44:56]

The patchset sent to the CQ was uploaded after l-g-t-m from
tedchoc@chromium.org, yusufo@chromium.org, isherman@chromium.org
Link to the patchset: https://codereview.chromium.org/2638653005/#ps20001
(title: "Add OWNERS file, per tedchoc's suggestion.")

[commit-bot: I haz the power, 2017-01-19 09:45:14]

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-01-19 10:58:00]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1484819095670940,
"parent_rev": "89bed32dfee51bcf0428012be7f09766543ac06a", "commit_rev":
"f8e7b55cbaf9147033f46e730a029347d3ae5baa"}

[commit-bot: I haz the power, 2017-01-19 10:58:38]

Description was changed from

==========
android: Record the source of account change notifications from GSA.

GSA can notify Chrome of an account change through either a service
call, or through a broadcast. This histogram records the number of
account changes of each type that Chrome sees.

This is intended to act as a migration tracker and sanity check as we
migrate from the service to the broadcast mechanism. The total count
should stay approximately the same, while the proportion of each type
changes.

BUG=614388
==========

to

==========
android: Record the source of account change notifications from GSA.

GSA can notify Chrome of an account change through either a service
call, or through a broadcast. This histogram records the number of
account changes of each type that Chrome sees.

This is intended to act as a migration tracker and sanity check as we
migrate from the service to the broadcast mechanism. The total count
should stay approximately the same, while the proportion of each type
changes.

BUG=614388

Review-Url: https://codereview.chromium.org/2638653005
Cr-Commit-Position: refs/heads/master@{#444705}
Committed:
https://chromium.googlesource.com/chromium/src/+/f8e7b55cbaf9147033f46e730a02...
==========

[commit-bot: I haz the power, 2017-01-19 10:58:39]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/f8e7b55cbaf9147033f46e730a02...