Adjust CSP of feedback extension

Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies, that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, becasue they expect resources to load from `file:` and `chrome:` and are using "*", which was addressed in CL
mentioned above.
Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/0e84bb1695f941cb15c99abc51970db0c65dc910
Cr-Commit-Position: refs/heads/master@{#411189}

[Sergey Shekyan, 2016-08-10 21:20:00]

Description was changed from

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, which was addressed in CL
mentioned above. Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
==========

to

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, which was addressed in CL
mentioned above. Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Sergey Shekyan, 2016-08-10 21:30:08]

Description was changed from

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, which was addressed in CL
mentioned above. Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies, that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, becasue they expect resources to load
from `file:` and `chrome:` and are using "*", which was addressed in CL
mentioned above. 
Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[jww, 2016-08-10 22:51:13]

On 2016/08/10 21:30:08, Sergey Shekyan wrote:
> Description was changed from
> 
> ==========
> Adjust CSP of feedback extension
> 
> Recent change to Content Security Policy matching algorithm for
> "*" source-expression at https://codereview.chromium.org/2209113002/
> requires to adjust policies that use "*" and assume that it will match
> resource loads from schemes other than http/https/ws/wss.
> Two extension manifests had to be changed, which was addressed in CL
> mentioned above. Unfortunately, I forgot to update style-src directive to
> also allow resources from `chrome:` scheme. This patch set fixes that.
> 
> BUG=633044
> mailto:R=jww@chromium.org
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> ==========
> 
> to
> 
> ==========
> Adjust CSP of feedback extension
> 
> Recent change to Content Security Policy matching algorithm for
> "*" source-expression at https://codereview.chromium.org/2209113002/
> requires to adjust policies, that use "*" and assume that it will match
> resource loads from schemes other than http/https/ws/wss.
> Two extension manifests had to be changed, becasue they expect resources to
load
> from `file:` and `chrome:` and are using "*", which was addressed in CL
> mentioned above. 
> Unfortunately, I forgot to update style-src directive to
> also allow resources from `chrome:` scheme. This patch set fixes that.
> 
> BUG=633044
> mailto:R=jww@chromium.org
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
> ==========

lgtm, thanks Sergey!

[Sergey Shekyan, 2016-08-10 22:51:36]

The CQ bit was checked by shekyan@gmail.com

[commit-bot: I haz the power, 2016-08-10 22:52:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jww, 2016-08-10 22:54:27]

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

[jww, 2016-08-10 22:54:27]

thestig@chromium.org: Can you owner review? Thanks!

[Lei Zhang, 2016-08-10 22:59:42]

lgtm

[commit-bot: I haz the power, 2016-08-10 23:52:17]

Description was changed from

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies, that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, becasue they expect resources to load
from `file:` and `chrome:` and are using "*", which was addressed in CL
mentioned above. 
Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies, that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, becasue they expect resources to load
from `file:` and `chrome:` and are using "*", which was addressed in CL
mentioned above. 
Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-08-10 23:52:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-10 23:54:11]

Description was changed from

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies, that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, becasue they expect resources to load
from `file:` and `chrome:` and are using "*", which was addressed in CL
mentioned above. 
Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Adjust CSP of feedback extension

Recent change to Content Security Policy matching algorithm for
"*" source-expression at https://codereview.chromium.org/2209113002/
requires to adjust policies, that use "*" and assume that it will match
resource loads from schemes other than http/https/ws/wss.
Two extension manifests had to be changed, becasue they expect resources to load
from `file:` and `chrome:` and are using "*", which was addressed in CL
mentioned above.
Unfortunately, I forgot to update style-src directive to
also allow resources from `chrome:` scheme. This patch set fixes that.

BUG=633044
R=jww@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/0e84bb1695f941cb15c99abc51970db0c65dc910
Cr-Commit-Position: refs/heads/master@{#411189}
==========

[commit-bot: I haz the power, 2016-08-10 23:54:12]

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