Bug 157100 – CSP: Add tests for setting allowContentSecurityPolicySourceStarToMatchAnyProtocol

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Daniel Bates]

We should add tests for the setting allowContentSecurityPolicySourceStarToMatchAnyProtocol to ensure that we do not regress its behavior. This setting was added as part of the patch for bug #157005. Although this setting is likely to be short lived it is straightforward to test this setting and should this setting turn out to not be short lived the presence of tests will ensure that we do not regress its behavior.

[Daniel Bates]

Created attachment 277532 [details]
Layout Tests

[Brent Fulgham]

Comment on attachment 277532 [details]
Layout Tests

View in context: https://bugs.webkit.org/attachment.cgi?id=277532&action=review

Looks good! r=me.

> LayoutTests/platform/wk2/TestExpectations:622
> +fast/dom/HTMLImageElement/image-with-blob-url-allowed-by-csp-img-src-star-with-AllowContentSecurityPolicySourceStarToMatchAnyProtocol-enabled.html

Can you please add the usual bugzilla link here for the WKTR lacking drag support?

[Daniel Bates]

(In reply to comment #2)
> [...]
> Can you please add the usual bugzilla link here for the WKTR lacking drag
> support?

Notice that this test is in the list of tests that fail because of a lack of support for eventSender.beginDragWithFiles() in WebKitTestRunner as mentioned in the comment at the start of this list, <http://trac.webkit.org/browser/trunk/LayoutTests/platform/wk2/TestExpectations?rev=199752#L617>.

[Daniel Bates]

Comment on attachment 277532 [details]
Layout Tests

Clearing flags on attachment: 277532

Committed r200155: <http://trac.webkit.org/changeset/200155>

[Daniel Bates]

All reviewed patches have been landed.  Closing bug.