|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
58 bytes, text/x-review-board-request
|Details | Review|
We currently parse "http://" and "ftp://" as valid URLs. They are not. See https://url.spec.whatwg.org/#hostname-state, special schemes (https://url.spec.whatwg.org/#special-scheme) cannot have empty hosts. Chrome agrees with the spec here.
I'm not sure if this is the right place to do this check, or if I should do it in nsURLParsers.cpp. nsURLParsers.cpp seems to be a bit more general.
We tried to fix this a few months ago, but it got backed out because it triggered a regression in the HTML form validator. I'll try to land it as soon as the form bug gets fixed.