Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area. It's not obvious it's a good
idea to convert the wildcard-patterns within that to RAPPOR domains.

BUG=639503
Committed: https://crrev.com/34cf29bff5e0b6375e356a945a5ce12d477f294c
Cr-Commit-Position: refs/heads/master@{#417198}

[tommycli, 2016-09-03 00:35:01]

Description was changed from

==========
Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area.

BUG=639503
==========

to

==========
Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area. It's not obvious it's a good
idea to convert the wildcard-patterns within that to RAPPOR domains.

BUG=639503
==========

[tommycli, 2016-09-03 00:35:01]

tommycli@chromium.org changed reviewers:
+ groby@chromium.org

[tommycli, 2016-09-03 00:36:04]

tommycli@chromium.org changed reviewers:
+ holte@chromium.org

[tommycli, 2016-09-03 00:36:05]

groby, holte: PTAL, thanks!

[groby-ooo-7-16, 2016-09-03 01:08:51]

lgtm

[Steven Holte, 2016-09-06 20:28:45]

lgtm

[tommycli, 2016-09-06 20:47:30]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2016-09-06 20:48:45]

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-09-06 20:57:51]

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

[commit-bot: I haz the power, 2016-09-06 20:57:53]

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

[tommycli, 2016-09-08 02:15:10]

tommycli@chromium.org changed reviewers:
+ thestig@chromium.org

[tommycli, 2016-09-08 02:15:11]

thestig: PTAL and cq if good enough? thanks!

[Lei Zhang, 2016-09-08 03:56:07]

The CQ bit was checked by thestig@chromium.org

[Lei Zhang, 2016-09-08 03:56:07]

lgtm

[commit-bot: I haz the power, 2016-09-08 03:56:22]

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-09-08 05:46:43]

Description was changed from

==========
Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area. It's not obvious it's a good
idea to convert the wildcard-patterns within that to RAPPOR domains.

BUG=639503
==========

to

==========
Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area. It's not obvious it's a good
idea to convert the wildcard-patterns within that to RAPPOR domains.

BUG=639503
==========

[commit-bot: I haz the power, 2016-09-08 05:46:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-08 05:49:00]

Description was changed from

==========
Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area. It's not obvious it's a good
idea to convert the wildcard-patterns within that to RAPPOR domains.

BUG=639503
==========

to

==========
Plugin Power Saver Tiny: Add RAPPOR metric for new ALLOW exceptions.

This catches new ALLOW exceptions added via the broken puzzle piece
in the Omnibox and via the Website Settings dropdown.

It doesn't catch ALLOW exceptions added manually via the
chrome://settings Content Settings area. It's not obvious it's a good
idea to convert the wildcard-patterns within that to RAPPOR domains.

BUG=639503

Committed: https://crrev.com/34cf29bff5e0b6375e356a945a5ce12d477f294c
Cr-Commit-Position: refs/heads/master@{#417198}
==========

[commit-bot: I haz the power, 2016-09-08 05:49:01]

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