[Extensions] Don't show a blocked action for webRequest on subframes

[Extensions] Don't show a blocked action for webRequest on subframes

Click-To-Script blocks webRequest on sites without permission, but we
currently don't have any mechanism of granting subframe origins. As
such, we don't count blocking subframe actions as a reason to show the
blocked action indicator in the case of content scripts running on
subframes. We should do the same for webRequest actions.

BUG=660901
Committed: https://crrev.com/eb53f1445275265a2b81112a69a5205004b709d3
Cr-Commit-Position: refs/heads/master@{#428787}

[Devlin, 2016-10-29 00:32:57]

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

[commit-bot: I haz the power, 2016-10-29 00:33:31]

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-29 01:25:40]

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

[commit-bot: I haz the power, 2016-10-29 01:25:41]

Dry run: This issue passed the CQ dry run.

[Devlin, 2016-10-31 17:53:21]

Description was changed from

==========
webrequest
BUG=
==========

to

==========
[Extensions] Don't show a blocked action for webRequest on subframes

Click-To-Script blocks webRequest on sites without permission, but we
currently don't have any mechanism of granting subframe origins. As
such, we don't count blocking subframe actions as a reason to show the
blocked action indicator in the case of content scripts running on
subframes. We should do the same for webRequest actions.

BUG=660901
==========

[Devlin, 2016-10-31 17:53:42]

rdevlin.cronin@chromium.org changed reviewers:
+ asargent@chromium.org

[Devlin, 2016-10-31 17:53:42]

Antony, can you take a look?

[asargent_no_longer_on_chrome, 2016-10-31 18:46:22]

lgtm

[Devlin, 2016-10-31 19:03:54]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-10-31 19:04:17]

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-31 20:27:17]

Description was changed from

==========
[Extensions] Don't show a blocked action for webRequest on subframes

Click-To-Script blocks webRequest on sites without permission, but we
currently don't have any mechanism of granting subframe origins. As
such, we don't count blocking subframe actions as a reason to show the
blocked action indicator in the case of content scripts running on
subframes. We should do the same for webRequest actions.

BUG=660901
==========

to

==========
[Extensions] Don't show a blocked action for webRequest on subframes

Click-To-Script blocks webRequest on sites without permission, but we
currently don't have any mechanism of granting subframe origins. As
such, we don't count blocking subframe actions as a reason to show the
blocked action indicator in the case of content scripts running on
subframes. We should do the same for webRequest actions.

BUG=660901
==========

[commit-bot: I haz the power, 2016-10-31 20:27:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-31 20:29:45]

Description was changed from

==========
[Extensions] Don't show a blocked action for webRequest on subframes

Click-To-Script blocks webRequest on sites without permission, but we
currently don't have any mechanism of granting subframe origins. As
such, we don't count blocking subframe actions as a reason to show the
blocked action indicator in the case of content scripts running on
subframes. We should do the same for webRequest actions.

BUG=660901
==========

to

==========
[Extensions] Don't show a blocked action for webRequest on subframes

Click-To-Script blocks webRequest on sites without permission, but we
currently don't have any mechanism of granting subframe origins. As
such, we don't count blocking subframe actions as a reason to show the
blocked action indicator in the case of content scripts running on
subframes. We should do the same for webRequest actions.

BUG=660901

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

[commit-bot: I haz the power, 2016-10-31 20:29:46]

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