Replace frame-src with child-src in WebUI CSP

Replace frame-src with child-src in WebUI CSP

"frame-src" is deprecated, and is replaced by "child-src".

BUG=81636, 336788
Committed: https://crrev.com/d6eb7796599ed3880244b2f292dee17a3bb8722a
Cr-Commit-Position: refs/heads/master@{#397869}

[wychen, 2016-05-27 23:16:46]

Description was changed from

==========
Replace frame-src with child-src in WebUI CSP

BUG=
==========

to

==========
Replace frame-src with child-src in WebUI CSP

BUG=81636
==========

[wychen, 2016-05-27 23:18:02]

Description was changed from

==========
Replace frame-src with child-src in WebUI CSP

BUG=81636
==========

to

==========
Replace frame-src with child-src in WebUI CSP

BUG=81636, 336788
==========

[wychen, 2016-05-28 00:27:06]

Description was changed from

==========
Replace frame-src with child-src in WebUI CSP

BUG=81636, 336788
==========

to

==========
Replace frame-src with child-src in WebUI CSP

"frame-src" is deprecated, and is replaced by "child-src".

BUG=81636, 336788
==========

[wychen, 2016-05-28 00:29:15]

wychen@chromium.org changed reviewers:
+ avi@chromium.org, dbeam@chromium.org, huangs@chromium.org,
mdjones@chromium.org, pfeldman@chromium.org

[wychen, 2016-05-28 00:29:35]

PTAL

[wychen, 2016-05-28 00:32:41]

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

[Avi (use Gerrit), 2016-05-28 00:50:11]

lgtm

[huangs, 2016-05-30 14:57:03]

Not sure why I'm on this review.  Anyway, "frame-src" still appears in some test
files, don't know if they need to be updated:

https://code.google.com/p/chromium/codesearch#search/&q=%22frame-src%22&sq=pa...

[wychen, 2016-05-31 07:46:34]

wychen@chromium.org changed reviewers:
- huangs@chromium.org

[wychen, 2016-05-31 07:46:55]

wychen@chromium.org changed reviewers:
+ huangs@chromium.org

[wychen, 2016-05-31 07:54:19]

On 2016/05/30 14:57:03, huangs wrote:
> Not sure why I'm on this review.  Anyway, "frame-src" still appears in some
test
> files, don't know if they need to be updated:
> 
>
https://code.google.com/p/chromium/codesearch#search/&q=%22frame-src%22&sq=pa...

Thanks for your comment. I added you for local_ntp_source.* ownership review.

There's one occurance in src/content/browser/site_per_process_browsertest.cc
after refining the query:
https://code.google.com/p/chromium/codesearch#search/&q=%5Cbframe-src%20test%...
It's not about WebUI specifically, so it's out of scope for this CL.

I'm not sure if we have a plan to replace all "frame-src" directives, but one
piece at a time.

[huangs, 2016-05-31 14:54:02]

Ah okay. LGTM for those files then.

[mdjones, 2016-05-31 16:13:02]

lgtm

[Tom Sepez, 2016-05-31 16:14:46]

lgtm

[Dan Beam, 2016-05-31 19:16:24]

lgtm

[wychen, 2016-06-03 23:34:29]

The CQ bit was checked by wychen@chromium.org

[commit-bot: I haz the power, 2016-06-03 23:35:24]

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

[commit-bot: I haz the power, 2016-06-04 00:53:06]

Description was changed from

==========
Replace frame-src with child-src in WebUI CSP

"frame-src" is deprecated, and is replaced by "child-src".

BUG=81636, 336788
==========

to

==========
Replace frame-src with child-src in WebUI CSP

"frame-src" is deprecated, and is replaced by "child-src".

BUG=81636, 336788
==========

[commit-bot: I haz the power, 2016-06-04 00:53:07]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-04 00:54:22]

Description was changed from

==========
Replace frame-src with child-src in WebUI CSP

"frame-src" is deprecated, and is replaced by "child-src".

BUG=81636, 336788
==========

to

==========
Replace frame-src with child-src in WebUI CSP

"frame-src" is deprecated, and is replaced by "child-src".

BUG=81636, 336788

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

[commit-bot: I haz the power, 2016-06-04 00:54:24]

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