Bug 217170

Summary: Copy host from base file URL
Product: WebKit Reporter: Anne van Kesteren <annevk>
Component: DOMAssignee: Alex Christensen <achristensen>
Status: RESOLVED FIXED    
Severity: Normal CC: achristensen, beidson, benjamin, cdumez, clopez, cmarcelo, ews-watchlist, webkit-bug-importer, youennf
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch beidson: review+

Anne van Kesteren
Reported 2020-10-01 03:00:46 PDT
See https://github.com/whatwg/url/pull/544 and new tests at https://github.com/web-platform-tests/wpt/pull/25716. Safari does not copy the host from the base URL for file URLs.
Attachments
Patch (36.87 KB, patch)
2020-10-01 15:23 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (40.14 KB, patch)
2020-10-01 16:37 PDT, Alex Christensen 		beidson: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2020-10-01 15:23:51 PDT
Created attachment 410276 [details] Patch
EWS Watchlist
Comment 2 2020-10-01 15:24:36 PDT
This patch modifies the imported WPT tests. Please ensure that any changes on the tests (not coming from a WPT import) are exported to WPT. Please see https://trac.webkit.org/wiki/WPTExportProcess
Alex Christensen
Comment 3 2020-10-01 16:37:33 PDT
Created attachment 410281 [details] Patch
Alex Christensen
Comment 4 2020-10-01 23:13:45 PDT
Comment on attachment 410281 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410281&action=review > Source/WTF/wtf/URLParser.cpp:1563 > + copyURLPartsUntil(base, URLPart::SchemeEnd, c, nonUTF8QueryEncoding); Note: this code does not have a corresponding case that copies until URLPart::PortEnd because we are transitioning to State::FileHost and we don't want to copy the host of the base URL here. The other cases do if there is one.
Alex Christensen
Comment 5 2020-10-02 13:59:23 PDT
http://trac.webkit.org/r267896
Radar WebKit Bug Importer
Comment 6 2020-10-02 14:00:25 PDT
<rdar://problem/69895958>
Note You need to log in before you can comment on or make changes to this bug.