Created attachment 408078 [details] Testcase This is a regression from bug 520848, pointed out in bug 508466 comment 17. Testcase attached. The key issue is the removal of the weight += listSel->CalcWeight() call in the old !IsSinglePseudoClass(*listSel) branch under (!nsCSSPseudoClasses::IsPseudoClass(pseudoClassList->mAtom)).
Oh, and we're no longer dropping rules with multiple pseudo-elements, though at least I suspect we're not matching them...
Created attachment 408089 [details] [diff] [review] Fix
Attachment #408089 - Flags: review?(zweinberg)
9 years ago
Comment on attachment 408089 [details] [diff] [review] Fix Looks good. CSS2.1 does say that pseudo-elements contribute to the 'd' component of the specificity (== the low byte of the weight) but I cannot presently think of a situation where it matters, and maybe that should be a follow-up bug.
Attachment #408089 - Flags: review?(zweinberg) → review+
Yeah, I can't think of a situation where it matters either, until we support selectors with more than one pseudo-element in them. And even then it might not.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
9 years ago
You need to log in before you can comment on or make changes to this bug.