Bug 265467
| Summary: | Stop requiring JSONP for layout tests results | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Sam Sneddon [:gsnedders] <gsnedders> |
| Component: | Tools / Tests | Assignee: | Sam Sneddon [:gsnedders] <gsnedders> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | ap, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=265606 | ||
| Bug Depends on: | |||
| Bug Blocks: | 265468 | ||
Sam Sneddon [:gsnedders]
Almost all of the call sites already optionally strip the JSONP wrapper if it exists, we should fix the rest and then drop the JSONP wrapper.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Sam Sneddon [:gsnedders]
Pull request: https://github.com/WebKit/WebKit/pull/21009
Alexey Proskuryakov
> drop the JSONP wrapper
Is there a reason why it's harmful? If we are to remove it, we need to think about security implications.
Alexey Proskuryakov
Will ask in the other bug.
EWS
Committed 271309@main (8a12b0af6759): <https://commits.webkit.org/271309@main>
Reviewed commits have been landed. Closing PR #21009 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/118955380>