RESOLVED FIXED161837
Optimize URLParser performance 
https://bugs.webkit.org/show_bug.cgi?id=161837
Summary Optimize URLParser performance
Alex Christensen
Reported 2016-09-09 23:06:12 PDT
Optimize URLParser performance
Attachments
Patch (4.99 KB, patch)
2016-09-09 23:09 PDT, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (6.73 KB, patch)
2016-09-10 16:42 PDT, Alex Christensen 		beidson: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2016-09-09 23:09:42 PDT
Created attachment 288482 [details] Patch
Alex Christensen
Comment 2 2016-09-10 16:42:25 PDT
Created attachment 288509 [details] Patch
Brady Eidson
Comment 3 2016-09-12 10:37:40 PDT
Comment on attachment 288509 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=288509&action=review > Source/WebCore/platform/URLParser.cpp:137 > + const uint16_t ftpPort = 21; > + const uint16_t gopherPort = 70; > + const uint16_t httpPort = 80; > + const uint16_t httpsPort = 433; > + const uint16_t wsPort = 80; > + const uint16_t wssPort = 433; static const > Tools/TestWebKitAPI/Tests/WebCore/URLParser.cpp:489 > + checkURL("hTtP://host:80", {"http", "", "", "host", 0, "/", "", "", "http://host/"}); If this test is worth adding, then similar mixed case tests are worth adding for *all* known schemes.
Alex Christensen
Comment 4 2016-09-12 11:09:17 PDT
http://trac.webkit.org/changeset/205812
Note You need to log in before you can comment on or make changes to this bug.