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>
Status: RESOLVED FIXED    
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   
Attachments:
Description Flags
Patch
none
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]
Patch
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]
Patch

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]
Patch

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
<rdar://problem/35567451>