Bug 208801 – Fix WPT test redirect-mode.any.html

• Component: New Bugs
• Status: RESOLVED FIXED

[Rob Buis]

Fix WPT test redirect-mode.any.html by also specifying the server port, this
is needed for WebKit local test runs.

[Rob Buis]

Created attachment 393017 [details]
Patch

[youenn fablet]

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]

Created attachment 393020 [details]
Patch

[Rob Buis]

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]

The commit-queue encountered the following flaky tests while processing attachment 393020 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 160571 (authors: [e-mail], [e-mail], and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 393020 [details]
Patch

Clearing flags on attachment: 393020

Committed r258144: <https://trac.webkit.org/changeset/258144>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/60224557>