[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

Adds UMA metrics for tracking "Turn Off" and "Ok, got it" buttons in Auto sign-in first run experience for the CM API.

BUG=610975
Committed: https://crrev.com/b53d8f6104063f23f369f0a2c90aecd0b048be1f
Cr-Commit-Position: refs/heads/master@{#392898}

[melandory, 2016-05-11 10:52:50]

melandory@chromium.org changed reviewers:
+ vasilii@chromium.org

[melandory, 2016-05-11 10:52:51]

PTAL, thanks!

[melandory, 2016-05-11 10:57:52]

Patchset #1 (id:1) has been deleted

[melandory, 2016-05-11 10:58:02]

Patchset #1 (id:20001) has been deleted

[vasilii, 2016-05-11 11:09:53]

The decryption is unclear. It should be something like "Add UMA metrics...".

Why don't you track "No decision"?

[melandory, 2016-05-11 11:15:00]

On 2016/05/11 11:09:53, vasilii wrote:
> The decryption is unclear. It should be something like "Add UMA metrics...".
> 
> Why don't you track "No decision"?
I track turn off decision. But if by no, you mean tracking when user dismissed
the dialog, then I don't do it because it will be a bit trickier in comparison
to tracking "ok" and "turn off", so I want to do it more careful in separate
patch.

[melandory, 2016-05-11 11:16:28]

Description was changed from

==========
[Smart Lock] Tracks "NO" and "OK" buttons clicks for Auto-signin FRE.

BUG=610975
==========

to

==========
[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

Adds UMA metrics for tracking "Turn Off" and "Ok, got it" buttons in Auto
sign-in first run experience for the CM API.

BUG=610975
==========

[melandory, 2016-05-11 11:16:45]

On 2016/05/11 11:09:53, vasilii wrote:
> The decryption is unclear. It should be something like "Add UMA metrics...".
Done.
> 
> Why don't you track "No decision"?

[vasilii, 2016-05-11 11:17:06]

lgtm

[melandory, 2016-05-11 11:19:05]

The CQ bit was checked by melandory@chromium.org

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

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1968013002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1968013002/40001

[commit-bot: I haz the power, 2016-05-11 11:22:41]

Description was changed from

==========
[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

Adds UMA metrics for tracking "Turn Off" and "Ok, got it" buttons in Auto
sign-in first run experience for the CM API.

BUG=610975
==========

to

==========
[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

Adds UMA metrics for tracking "Turn Off" and "Ok, got it" buttons in Auto
sign-in first run experience for the CM API.

BUG=610975
==========

[commit-bot: I haz the power, 2016-05-11 11:22:42]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-05-11 11:25:23]

Description was changed from

==========
[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

Adds UMA metrics for tracking "Turn Off" and "Ok, got it" buttons in Auto
sign-in first run experience for the CM API.

BUG=610975
==========

to

==========
[Smart Lock] Adds UMA Metrics for buttons clicks in Auto-signin FRE.

Adds UMA metrics for tracking "Turn Off" and "Ok, got it" buttons in Auto
sign-in first run experience for the CM API.

BUG=610975

Committed: https://crrev.com/b53d8f6104063f23f369f0a2c90aecd0b048be1f
Cr-Commit-Position: refs/heads/master@{#392898}
==========

[commit-bot: I haz the power, 2016-05-11 11:25:24]

Patchset 1 (id:??) landed as
https://crrev.com/b53d8f6104063f23f369f0a2c90aecd0b048be1f
Cr-Commit-Position: refs/heads/master@{#392898}