Return early in SelectorChecker::checkOne() if selector.isAttributeSelector() is true and the selector matched. Previously, we were only returning early if the selector did not match and we were falling through if it did. This means that we ended up doing the 2 if() checks below in the method unnecessarily: - selector.m_match == CSSSelector::PseudoClass - selector.m_match == CSSSelector::PseudoElement
Created attachment 238142 [details] Patch
Comment on attachment 238142 [details] Patch Please do not optimize SelectorChecker, it is *completely* untested. The rule for SelectorChecker is now to make it correct no matter how slow "correct" is.
Comment on attachment 238142 [details] Patch Clearing flags on attachment: 238142 Committed r173646: <http://trac.webkit.org/changeset/173646>
All reviewed patches have been landed. Closing bug.