RESOLVED FIXED 208801
Fix WPT test redirect-mode.any.html 
https://bugs.webkit.org/show_bug.cgi?id=208801
Summary Fix WPT test redirect-mode.any.html
Rob Buis
Reported 2020-03-09 02:11:33 PDT
Fix WPT test redirect-mode.any.html by also specifying the server port, this is needed for WebKit local test runs.
Attachments
Patch (11.83 KB, patch)
2020-03-09 02:15 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Patch (11.83 KB, patch)
2020-03-09 03:44 PDT, Rob Buis 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2020-03-09 02:15:25 PDT
Created attachment 393017 [details] Patch
youenn fablet
Comment 2 2020-03-09 03:27:35 PDT
Comment on attachment 393017 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=393017&action=review > LayoutTests/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-mode.any.js:9 > + url.port = get_host_info().HTTP_PORT; Is the URL only HTTP? Should we consider HTTPS_PORT as well?
Rob Buis
Comment 3 2020-03-09 03:44:50 PDT
Created attachment 393020 [details] Patch
Rob Buis
Comment 4 2020-03-09 07:30:51 PDT
Comment on attachment 393017 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=393017&action=review >> LayoutTests/imported/w3c/web-platform-tests/fetch/api/redirect/redirect-mode.any.js:9 >> + url.port = get_host_info().HTTP_PORT; > > Is the URL only HTTP? Should we consider HTTPS_PORT as well? I think for https it would have needed the https extension in the file name. BTW Anne reviewed so I'll land this soon, not sure when it will get updated on wpt.fyi.
WebKit Commit Bot
Comment 5 2020-03-09 08:45:29 PDT
The commit-queue encountered the following flaky tests while processing attachment 393020 [details]: editing/spelling/spellcheck-async-remove-frame.html bug 160571 (authors: morrita@google.com, rniwa@webkit.org, and tony@chromium.org) The commit-queue is continuing to process your patch.
WebKit Commit Bot
Comment 6 2020-03-09 08:46:05 PDT
Comment on attachment 393020 [details] Patch Clearing flags on attachment: 393020 Committed r258144: <https://trac.webkit.org/changeset/258144>
WebKit Commit Bot
Comment 7 2020-03-09 08:46:06 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2020-03-09 08:47:18 PDT
<rdar://problem/60224557>
Note You need to log in before you can comment on or make changes to this bug.