RESOLVED FIXED 21052
Generalize id selector special case for querySelectorAll 
https://bugs.webkit.org/show_bug.cgi?id=21052
Summary Generalize id selector special case for querySelectorAll
David Smith
Reported 2008-09-24 02:05:46 PDT
Currently we special case only a raw id selector. Patch forthcoming shortly to expand this to any selector with an id selector as its rightmost element, at a small cost in special case speed. Changes results from: #title: 2ms h1#title: 55ms div #title: 55ms to: #title: 1ms (test variance? it shouldn't have sped up...) h1#title: 2ms div #title: 5ms
Attachments
Patch; Passes run-webkit-tests and doesn't regress http://ejohn.org/apps/selectortest/#target (4.06 KB, patch)
2008-09-24 02:21 PDT, David Smith 		timothy: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
David Smith
Comment 1 2008-09-24 02:21:09 PDT
Created attachment 23745 [details] Patch; Passes run-webkit-tests and doesn't regress http://ejohn.org/apps/selectortest/#target
David Smith
Comment 2 2008-09-25 13:56:47 PDT
Fixed in r36907
Note You need to log in before you can comment on or make changes to this bug.