Needs to ignore repeated slashes in file URL

[watilde]

Hello!

Since the file state doesn't have a way to jump to the special authority slashes state, once the code enters into the file state, repeated slashes in the beginning of a file URL(e.g. file://///foo/bar should be file:///foo/bar) can't be ignored.

To cover the case, it could be done by the following steps:

1. In scheme state, stop jumping to file state directly before checking special authority slashes state.
2. In special authority ignore slashes state, add file state check.
3. In scheme state, add file state check before jumping to special relative or authority state.

Resources:

• In the bs file, it can be like this: watilde@b587151
• Here is an example impl: watilde/node@247f960

What do you think?

[annevk]

It does seem that Chrome and Edge have this behavior, while Firefox and Safari (likely changed I suppose) do not.

I would support changing this based on that. We'll need new tests as well.

[watilde]

@annevk Thanks for your comment!

Then I will try reading external/google-url(correct?) impl as a reference, and write some tests to WPT. To make a PR to WPT, could I possibility make a PR to this repo with this commit before that?

[annevk]

Yeah, you can do the PRs independently as you wish. We'll link them all up before landing.

As for what to use as reference, google-url seems good, but testing Edge would be good too. As for what to test, switching between \ and / would be good, and whether the eventual non-//\ code point ends up as path or host. Not sure whether there is much else here.