Fix more URLParser quirks
Created attachment 288479 [details] Patch
Attachment 288479 [details] did not pass style-queue: ERROR: Source/WebCore/ChangeLog:8: You should remove the 'No new tests' and either add and list tests, or explain why no new tests were possible. [changelog/nonewtests] [5] Total errors found: 1 in 4 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 288481 [details] Patch
http://trac.webkit.org/changeset/205813
I guess the spec does say to always go to "special authority ignore slashes state" where '\' is accepted, possibly with a syntax violation. The only reason I needed to do anything special is because of my serialize-as-I-parse implementation. I don't think the spec needs to be changed here. I'll remove my comment. I do think the syntax violations are an implementation detail and shouldn't be in the spec. There are a few places where the spec has mistakes in how it reports syntax violations, notably in the ipv4 address parser. That's a different issue, though