Add UMA for signin through NTP content suggestions.

Add UMA for signin through NTP content suggestions.

BUG=636523
Committed: https://crrev.com/685346d8f50c8f04e81a4388e164203304f4d58c
Cr-Commit-Position: refs/heads/master@{#412554}

[PEConn, 2016-08-10 22:42:22]

peconn@chromium.org changed reviewers:
+ asvitkine@chromium.org, atwilson@chromium.org, bauerb@chromium.org,
dgn@chromium.org

[PEConn, 2016-08-10 22:42:22]

bauerb@chromium.org and dgn@chromium.org: Please review changes in:
chrome/android*

atwilson@chromium.org: Please review changes in:
signin_metrics.h

asvitkine@chromium.org: Please review changes in:
actions.xml
histograms.xml

[Alexei Svitkine (slow), 2016-08-11 05:50:04]

lgtm

[Bernhard Bauer, 2016-08-11 10:45:27]

lgtm

[PEConn, 2016-08-11 13:49:34]

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

[commit-bot: I haz the power, 2016-08-11 13:49:48]

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-08-11 14:02:45]

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

[commit-bot: I haz the power, 2016-08-11 14:02:46]

Dry run: 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_...)

[PEConn, 2016-08-11 15:36:40]

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

[commit-bot: I haz the power, 2016-08-11 15:37:00]

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-08-11 16:50:55]

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

[commit-bot: I haz the power, 2016-08-11 16:50:56]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[PEConn, 2016-08-15 16:11:42]

peconn@chromium.org changed reviewers:
+ pkotwicz@chromium.org
- dgn@chromium.org

[PEConn, 2016-08-15 16:12:10]

Description was changed from

==========
Add UMA for signin through NTP content suggestions.

BUG=636523
==========

to

==========
Add UMA for signin through NTP content suggestions.

BUG=636523
==========

[PEConn, 2016-08-15 16:12:10]

peconn@chromium.org changed reviewers:
+ torne@chromium.org
- pkotwicz@chromium.org

[PEConn, 2016-08-15 16:13:07]

@torne: Could I please get an LGTM for RecordUserAction.java, my change is
similar to what is already implemented in RecordHistogram.java

[PEConn, 2016-08-15 16:13:22]

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

[commit-bot: I haz the power, 2016-08-15 16:13:35]

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

[Torne, 2016-08-15 16:16:30]

base/android LGTM

[commit-bot: I haz the power, 2016-08-15 17:09:05]

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

[commit-bot: I haz the power, 2016-08-15 17:09:06]

Dry run: This issue passed the CQ dry run.

[PEConn, 2016-08-15 20:20:44]

The CQ bit was checked by peconn@chromium.org

[PEConn, 2016-08-15 20:20:46]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org,
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/2230183005/#ps40001
(title: "Do not record user actions in tests.")

[commit-bot: I haz the power, 2016-08-15 20:21:06]

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-08-15 20:29:32]

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

[commit-bot: I haz the power, 2016-08-15 20:29:34]

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...)

[PEConn, 2016-08-16 16:03:10]

peconn@chromium.org changed reviewers:
+ rogerta@chromium.org

[PEConn, 2016-08-16 16:03:11]

@rogerta: Could you please look at the changes in signin_metrics.cc

[PEConn, 2016-08-17 13:49:05]

@atwilson: Could you please look at the changes to signin_metrics?

[Andrew T Wilson (Slow), 2016-08-17 13:57:59]

On 2016/08/17 13:49:05, PEConn wrote:
> @atwilson: Could you please look at the changes to signin_metrics?

signin_metrics LGTM

[PEConn, 2016-08-17 14:02:01]

The CQ bit was checked by peconn@chromium.org

[commit-bot: I haz the power, 2016-08-17 14:02:18]

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-08-17 14:04:52]

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

[commit-bot: I haz the power, 2016-08-17 14:04:53]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[PEConn, 2016-08-17 15:24:30]

The CQ bit was checked by peconn@chromium.org

[PEConn, 2016-08-17 15:24:31]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org,
asvitkine@chromium.org, torne@chromium.org, atwilson@chromium.org
Link to the patchset: https://codereview.chromium.org/2230183005/#ps60001
(title: "Merge.")

[commit-bot: I haz the power, 2016-08-17 15:24:53]

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-08-17 16:27:50]

Description was changed from

==========
Add UMA for signin through NTP content suggestions.

BUG=636523
==========

to

==========
Add UMA for signin through NTP content suggestions.

BUG=636523
==========

[commit-bot: I haz the power, 2016-08-17 16:27:54]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-17 16:30:14]

Description was changed from

==========
Add UMA for signin through NTP content suggestions.

BUG=636523
==========

to

==========
Add UMA for signin through NTP content suggestions.

BUG=636523

Committed: https://crrev.com/685346d8f50c8f04e81a4388e164203304f4d58c
Cr-Commit-Position: refs/heads/master@{#412554}
==========

[commit-bot: I haz the power, 2016-08-17 16:30:15]

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