Plugin Power Saver: Remove ASK (left C2P). Replace with DETECT.

Plugin Power Saver: Remove ASK (left C2P). Replace with DETECT.

Patchset 1 is a straight revert of https://codereview.chromium.org/829113004.

Subsequent patches are modifications to the revert. Previously, we replaced ASK with BLOCK.

BUG=456248, 403800
Committed: https://crrev.com/54d9cc0c8fc905230aae59ccba4f7bc30c559aa9
Cr-Commit-Position: refs/heads/master@{#315629}

[tommycli, 2015-02-06 20:26:35]

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

[tommycli, 2015-02-06 20:28:33]

thestig: PTAL, thanks!

Patchset 1 is a straight revert of the old CL (which is also a revert).

Subsequent patchsets change ASK=>BLOCK to ASK=>DETECT.

Thanks!

[Lei Zhang, 2015-02-07 00:02:02]

I don't see any changes to actions.xml in patch set 3.

[tommycli, 2015-02-09 20:14:32]

On 2015/02/07 00:02:02, Lei Zhang wrote:
> I don't see any changes to actions.xml in patch set 3.

thestig:

Hey. 'actions.xml' is changed in Patchsets 1 and 2. 

Patchset 3 removes actions.xml from the list of changes.

You told me something about this topic on Friday, but I don't remember what that
was...

Thanks!

[Lei Zhang, 2015-02-10 03:20:05]

lgtm

I edited the commit msg a little.

[tommycli, 2015-02-10 17:54:09]

The CQ bit was checked by tommycli@chromium.org

[commit-bot: I haz the power, 2015-02-10 17:54:42]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/902223002/60001

[commit-bot: I haz the power, 2015-02-10 18:01:28]

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

[commit-bot: I haz the power, 2015-02-10 18:01:29]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[tommycli, 2015-02-10 18:38:29]

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

[tommycli, 2015-02-10 18:38:30]

tsepez: ptal render_messages.h

Enum is only used internally, and not as integer values, so should have minimal
security implications. Also this patch is basically a revert.

[Tom Sepez, 2015-02-10 19:25:17]

Messages LGTM

[tommycli, 2015-02-10 19:28:08]

The CQ bit was checked by tommycli@chromium.org

[tommycli, 2015-02-10 19:28:13]

tsepez: Thanks!

[commit-bot: I haz the power, 2015-02-10 19:28:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/902223002/60001

[commit-bot: I haz the power, 2015-02-10 20:32:44]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-02-10 20:33:08]

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