Bug 180220

Summary: WKURLSchemeHandler.request should include HTTPBody
Product: WebKit Reporter: Alex Christensen <achristensen>
Component: New BugsAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: beidson, ccorcos, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Alex Christensen
Reported 2017-11-30 14:46:31 PST
WKURLSchemeHandler.request should include HTTPBody
Attachments
Patch (21.30 KB, patch)
2017-11-30 14:48 PST, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (21.77 KB, patch)
2017-11-30 15:00 PST, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2017-11-30 14:48:31 PST
Created attachment 328034 [details] Patch
Alex Christensen
Comment 2 2017-11-30 14:52:51 PST
Comment on attachment 328034 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=328034&action=review > Source/WebKit/Shared/URLSchemeTaskParameters.cpp:42 > + if (request.httpBody()) { > + encoder << true; > + request.httpBody()->encode(encoder); This is similar to NetworkResourceLoadParameters::encode except we are encoding to the UIProcess, so we don't need to encode sandbox extensions like we do there.
Alex Christensen
Comment 3 2017-11-30 15:00:46 PST
Created attachment 328036 [details] Patch
Alex Christensen
Comment 4 2017-11-30 15:16:43 PST
http://trac.webkit.org/r225358
Radar WebKit Bug Importer
Comment 5 2017-11-30 15:18:42 PST
<rdar://problem/35784105>
Chet Corcos
Comment 6 2018-10-10 12:59:43 PDT
Hey there, how do I know if / when this makes it into an iOS release?
Note You need to log in before you can comment on or make changes to this bug.