Canvas rgb() color parsing still slow
Created attachment 50925 [details] Patch
Comment on attachment 50925 [details] Patch r=me, what's your testcase? and what's the improvement? --Oliver
(In reply to comment #2) > (From update of attachment 50925 [details]) > r=me, what's your testcase? and what's the improvement? Test case is Peacekeeper "Complex Graphics". Yields ~8% gain in Chromium/Win.
Comment on attachment 50925 [details] Patch Clearing flags on attachment: 50925 Committed r56155: <http://trac.webkit.org/changeset/56155>
All reviewed patches have been landed. Closing bug.
The committed patch didn't have reviewer name in ChangeLog.
The commit bot looks for and replaces "Reviewed by NOBODY(OOPS!)" with the correct information. Given the recent reviewer parsing added in http://trac.webkit.org/changeset/56137 it would be possible to finally write a minimal fix for https://bugs.webkit.org/show_bug.cgi?id=26927 Until then, please dont' change the Reviewed by NOBODY line.
(In reply to comment #7) > The commit bot looks for and replaces "Reviewed by NOBODY(OOPS!)" with the > correct information. > > Given the recent reviewer parsing added in > http://trac.webkit.org/changeset/56137 > it would be possible to finally write a minimal fix for > https://bugs.webkit.org/show_bug.cgi?id=26927 > > Until then, please dont' change the Reviewed by NOBODY line. (In reply to comment #7) > The commit bot looks for and replaces "Reviewed by NOBODY(OOPS!)" with the > correct information. > > Given the recent reviewer parsing added in > http://trac.webkit.org/changeset/56137 > it would be possible to finally write a minimal fix for > https://bugs.webkit.org/show_bug.cgi?id=26927 > > Until then, please dont' change the Reviewed by NOBODY line. Sorry about that. I must've deleted it by mistake. Should I land a patch to fix the ChangeLog?