Bug 265468 - Stop outputting JSONP for layout tests results
Summary: Stop outputting JSONP for layout tests results
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Sam Sneddon [:gsnedders]
URL:
Keywords: InRadar
Depends on: 265467 265606
Blocks:
  Show dependency tree / graph
 
Reported: 2023-11-28 10:54 PST by Sam Sneddon [:gsnedders]
Modified: 2024-03-06 04:51 PST (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sam Sneddon [:gsnedders] 2023-11-28 10:54:30 PST 
Once bug 265467 is fixed, we can start to fix what was planned in 2011 (c.f. https://commits.webkit.org/89351@main), and start storing/serving JSON (and not JSONP) everywhere.
Comment 1 Sam Sneddon [:gsnedders] 2023-11-28 11:21:12 PST 
Pull request: https://github.com/WebKit/WebKit/pull/21011
Comment 2 Radar WebKit Bug Importer 2023-12-05 10:55:15 PST 
<rdar://problem/119200588>
Comment 3 Sam Sneddon [:gsnedders] 2024-03-06 04:51:14 PST 
We can't actually do this, *because* we expect to be able to load full_results.json from file:///, adjacent to results.html, and any XHR/fetch there is considered cross-origin (and we have no way to opt-in to making full_results.json shareable with CORS).

This feels unfortunate, but we don't really have any better way of doing cross-origin access without CORS than JSONP, so our function call must remain (boo).