URLParser should parse IPv4 addresses
Created attachment 286510 [details] Patch
Comment on attachment 286510 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=286510&action=review efl build failure unrelated. > Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp:122 > +TEST_F(URLParserTest, ParserDifferences) These changes are because of the new URLParser being more standards compliant.
Comment on attachment 286510 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=286510&action=review >> Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp:122 >> +TEST_F(URLParserTest, ParserDifferences) > > These changes are because of the new URLParser being more standards compliant. Why is this comment in the patch review rather than in the test?
Comment on attachment 286510 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=286510&action=review >>> Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp:122 >>> +TEST_F(URLParserTest, ParserDifferences) >> >> These changes are because of the new URLParser being more standards compliant. > > Why is this comment in the patch review rather than in the test? I'll put a comment in the next patch. I don't have SVN on my phone :( There will be more differences and I'll put them here until we remove the old parser, then convert the tests to a new form because there will be no old parser to compare it with.
Comment on attachment 286510 [details] Patch Clearing flags on attachment: 286510 Committed r204701: <http://trac.webkit.org/changeset/204701>
All reviewed patches have been landed. Closing bug.
Is the URL parser ready enough to start using within fetch API?
(In reply to comment #7) > Is the URL parser ready enough to start using within fetch API? Not yet