Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Maybe ensure there is a path item after the Windows drive letter path item #193

Closed
zcorpan opened this issue Jan 3, 2017 · 1 comment
Closed

Comments

@zcorpan
Copy link
Member

zcorpan commented Jan 3, 2017

https://url.spec.whatwg.org/commit-snapshots/26caf4927f6c95ba4f56d701ecd8a52fdb664982/#path-state

Otherwise, if buffer is not a single-dot path segment, run these subsubsteps:​

If url’s scheme is "file", url’s path is empty, and buffer is a Windows drive letter, run these subsubsubsteps:​

If url’s host is non-null, syntax violation.

Set url’s host to null and replace the second code point in buffer with ":​".

This is a (platform-independent) Windows drive letter quirk.

Append buffer to url’s path.

From web-platform-tests/wpt#4382 (comment)

Parsing //d: with base file:///C:/a/b (or just parsing file:///d:) currently gives a single string "d:" in the path, so serializing does not include a trailing slash. Maybe it should?

http://software.hixie.ch/utilities/js/live-dom-viewer/saved/4769

No trailing slash:
Edge 13 and IE 9..11, Safari 9.1 and TP (WebKit 12603.1.16), Firefox Nightly 52.0a1 (2016-11-14) and 53.0a1 (2017-01-02) on Mac OS X, Chrome Canary 57.0.2970.0 on Mac OS X
With trailing slash:
Chrome 50.0 on Windows 10, Firefox 44.0 on Windows 10

(I don't know if there is a difference between platforms here for Chromium and Gecko, or if it changed between recent versions.)

@annevk
Copy link
Member

annevk commented Feb 2, 2017

Aligning with Edge seems good. Let's close this.

@annevk annevk closed this as completed Feb 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants