[webkit-reviews] review granted: [Bug 109429] CSP reports for blocked 'data:' URLs should report the scheme only. : [Attachment 187566] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Feb 11 10:35:54 PST 2013


Adam Barth <abarth at webkit.org> has granted Mike West <mkwst at chromium.org>'s
request for review:
Bug 109429: CSP reports for blocked 'data:' URLs should report the scheme only.
https://bugs.webkit.org/show_bug.cgi?id=109429

Attachment 187566: Patch
https://bugs.webkit.org/attachment.cgi?id=187566&action=review

------- Additional Comments from Adam Barth <abarth at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=187566&action=review


> Source/WebCore/page/ContentSecurityPolicy.cpp:1634
> +	   if (blockedURL.canSetHostOrPort())

I'd use isHierarchical rather than canSetHostOrPort, but I think they're the
same.


More information about the webkit-reviews mailing list