Remove addHTTPOriginIfNeeded calls since they are get requests and navigations and the spec [1] indicates that the Origin header should not be written out, making these calls no-ops. [1] https://fetch.spec.whatwg.org/#append-a-request-origin-header
Created attachment 393620 [details] Patch
Comment on attachment 393620 [details] Patch Clearing flags on attachment: 393620 Committed r258488: <https://trac.webkit.org/changeset/258488>
All reviewed patches have been landed. Closing bug.
<rdar://problem/60486281>
Reopening to attach new patch.
Created attachment 394865 [details] Patch
Comment on attachment 394865 [details] Patch Looks fine. Next time I suggest using a new bug report. I don’t think reopening is a good way to handle follow up even though I see a lot of people doing it. I think reopening is a good way to handle trying again rolling out or trying again if the first try did not successfully fix the bug.
Point taken!
Committed r259182: <https://trac.webkit.org/changeset/259182> All reviewed patches have been landed. Closing bug and clearing flags on attachment 394865 [details].