We can add performance tests for the Element.id getter and setter. If this patch looks good, I would like to add similar performance tests for other DOM attributes.
Created attachment 128158 [details] Patch
This is a pretty simplistic microbenchmark, but it seems useful.
Comment on attachment 128158 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=128158&action=review > PerformanceTests/Bindings/dom_attributes_id_getter.html:1 > +<html> Missing DOCTYPE. > PerformanceTests/Bindings/dom_attributes_id_setter.html:1 > +<html> Ditto.
Created attachment 128161 [details] patch for landing
(In reply to comment #3) > (From update of attachment 128158 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=128158&action=review > > > PerformanceTests/Bindings/dom_attributes_id_getter.html:1 > > +<html> > > Missing DOCTYPE. > > > PerformanceTests/Bindings/dom_attributes_id_setter.html:1 > > +<html> > > Ditto. Fixed. Thanks.
Comment on attachment 128161 [details] patch for landing Clearing flags on attachment: 128161 Committed r108477: <http://trac.webkit.org/changeset/108477>
Reverted r108477 for reason: The perf tests committed in r108748 covers the perf tests committed in r108477. So I'll roll out r108477. Committed r108782: <http://trac.webkit.org/changeset/108782>