RESOLVED FIXED 78220
Avoid unnecessary work when evaluating style sharing candidates. 
https://bugs.webkit.org/show_bug.cgi?id=78220
Summary Avoid unnecessary work when evaluating style sharing candidates.
Andreas Kling
Reported 2012-02-09 03:51:06 PST
CSSStyleSelector::canShareStyleWithElement() has gathered a fair bit of cruft, we can reorder parts of it, and avoid some bits altogether.
Attachments
Proposed patch (6.33 KB, patch)
2012-02-09 04:25 PST, Andreas Kling 		koivisto: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2012-02-09 04:25:10 PST
Created attachment 126277 [details] Proposed patch
Andreas Kling
Comment 2 2012-02-09 10:56:52 PST
Committed r107261: <http://trac.webkit.org/changeset/107261>
Julien Chaffraix
Comment 3 2012-02-09 13:05:07 PST
Reopening as I rolled the change out in r107279. This test started failing after the change: svg/W3C-SVG-1.1/styling-css-05-b.svg Visually, it looked like the wrong lang() selector was applied: the test assumes we apply the lang(fr) one but we were applying lang(en) after the change.
Andreas Kling
Comment 4 2012-02-13 02:24:06 PST
Relanded in <http://trac.webkit.org/changeset/107551> without breaking the lang() selector.
Note You need to log in before you can comment on or make changes to this bug.