We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
From @domenic in #260 (comment):
Note that there is no test coverage for the case introduced here: in if (this.url.host !== "" && this.url.host !== null) { this.parseError = true; this.url.host = ""; } the condition never evaluates to true so those two lines are never reached. Maybe a coverage gap, maybe a spec bug.
Note that there is no test coverage for the case introduced here: in
if (this.url.host !== "" && this.url.host !== null) { this.parseError = true; this.url.host = ""; }
the condition never evaluates to true so those two lines are never reached. Maybe a coverage gap, maybe a spec bug.
The text was updated successfully, but these errors were encountered:
File URL's parsing tests for path state
634175d
Tests windows drive letter quirk in the path state: https://url.spec.whatwg.org/#path-state Fixes whatwg/url#272 (whatwg/url#260 (comment))
Auto merge of #410 - valenting:win-file-letter, r=SimonSapin
c6284a2
Properly file URLs when the path starts with a windows drive letter This change imports tests from web-platform-tests/wpt@634175d corresponding to URL change whatwg/url#272 <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-url/410) <!-- Reviewable:end -->
No branches or pull requests
From @domenic in #260 (comment):
The text was updated successfully, but these errors were encountered: