RESOLVED FIXED10722
WebKit spends 5% of total time parsing WorldCup.svg in bad parsing code 
https://bugs.webkit.org/show_bug.cgi?id=10722
Summary WebKit spends 5% of total time parsing WorldCup.svg in bad parsing code
Eric Seidel (no email)
Reported 2006-09-04 01:34:28 PDT
WebKit spends 5% of total time parsing WorldCup.svg in bad parsing code These two (misdesigned) lines in SVGPathParser::parseSVG() account for over 5% of total time: d = d.replace(',', ' '); d = d.simplifyWhiteSpace(); No need to do either. Just encoding that logic into the parser and do a single walk over the string would be much faster. Somewhere there is a darin thinking "i told you so..." ;)
Attachments
Patch ignoring whitespace (5.76 KB, patch)
2006-09-05 02:16 PDT, Eric Seidel (no email) 		aroben: review+
DetailsFormatted DiffDiff
full patch (26.97 KB, patch)
2006-09-05 02:16 PDT, Eric Seidel (no email) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2006-09-05 02:16:35 PDT
Created attachment 10398 [details] Patch ignoring whitespace
Eric Seidel (no email)
Comment 2 2006-09-05 02:16:57 PDT
Created attachment 10399 [details] full patch
Adam Roben (:aroben)
Comment 3 2006-09-05 20:16:47 PDT
Comment on attachment 10398 [details] Patch ignoring whitespace r=me, and ditto for the w/ whitespace patch, assuming there really are no other differences :-)
Note You need to log in before you can comment on or make changes to this bug.