Bug 179250

Summary: Requests handled by Service Worker should not go through preflighting
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebCore Misc.Assignee: youenn fablet <youennf>
Severity: Normal CC: achristensen, beidson, buildbot, cdumez, commit-queue, dbates, japhet, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
Build fix none

Description youenn fablet 2017-11-03 11:27:17 PDT
DocumentThreadableLoader is using preflight for any non simple CORS request.
In case request is handled by SW, no preflighting should happen.
If SW is not handling the request, preflighting should happen.
Comment 1 youenn fablet 2017-11-03 11:45:09 PDT
Created attachment 325928 [details]
Comment 2 youenn fablet 2017-11-03 14:45:25 PDT
Created attachment 325964 [details]
Build fix
Comment 3 Alex Christensen 2017-11-03 15:11:11 PDT
Do we intend to always not allow requests that would require preflight?  Why?
Comment 4 Alex Christensen 2017-11-03 15:22:16 PDT
Comment on attachment 325928 [details]

We really ought to move preflight to the right place.
Comment 5 WebKit Commit Bot 2017-11-03 16:09:31 PDT
Comment on attachment 325928 [details]

Clearing flags on attachment: 325928

Committed r224439: <https://trac.webkit.org/changeset/224439>
Comment 6 WebKit Commit Bot 2017-11-03 16:09:33 PDT
All reviewed patches have been landed.  Closing bug.
Comment 7 Radar WebKit Bug Importer 2017-11-15 12:23:52 PST