Record a user action when a content intent is sent

Record a user action when a content intent is sent

If the user activates a content detector for a phone number,
address, or email address, record a user action when the intent
is sent.

BUG=657852
Committed: https://crrev.com/e2cf786bb19426e44fe22a36fb8d509ea04d1e62
Cr-Commit-Position: refs/heads/master@{#426801}

[Theresa, 2016-10-20 21:06:43]

twellington@chromium.org changed reviewers:
+ aelias@chromium.org

[Theresa, 2016-10-20 21:06:44]

ptal

[aelias_OOO_until_Jul13, 2016-10-20 21:59:00]

What metric are you planning to use as the denominator for this?  Usually, it's
better to express metrics as some kind of percentage.

[Theresa, 2016-10-20 22:22:28]

On 2016/10/20 21:59:00, aelias wrote:
> What metric are you planning to use as the denominator for this?  Usually,
it's
> better to express metrics as some kind of percentage.

We're trying to determine the % of users that will be impacted by removing the
current content detector implementation, so the denominator is total # or users.
The user actions dashboard automatically displays % of users for which actions
are logged.

[aelias_OOO_until_Jul13, 2016-10-20 22:27:48]

OK, lgtm.

[Theresa, 2016-10-20 22:40:49]

twellington@chromium.org changed reviewers:
+ rkaplow@chromium.org

[Theresa, 2016-10-20 22:40:50]

+rkaplow@ for actions.xml

[Theresa, 2016-10-20 22:40:55]

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

[commit-bot: I haz the power, 2016-10-20 22:41:42]

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-21 00:12:57]

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

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

Dry run: This issue passed the CQ dry run.

[rkaplow, 2016-10-21 15:24:05]

lgtm

[Theresa, 2016-10-21 15:26:32]

The CQ bit was checked by twellington@chromium.org

[commit-bot: I haz the power, 2016-10-21 15:26:50]

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-21 15:51:00]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-21 16:04:47]

Description was changed from

==========
Record a user action when a content intent is sent

If the user activates a content detector for a phone number,
address, or email address, record a user action when the intent
is sent.

BUG=657852
==========

to

==========
Record a user action when a content intent is sent

If the user activates a content detector for a phone number,
address, or email address, record a user action when the intent
is sent.

BUG=657852

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

[commit-bot: I haz the power, 2016-10-21 16:04:48]

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