Chromium Code Reviews| Index: Source/core/css/SelectorChecker.cpp |
| diff --git a/Source/core/css/SelectorChecker.cpp b/Source/core/css/SelectorChecker.cpp |
| index 099a94cd6a4073d6787053d246c48cf3df6305a2..5e6120d9b62cc8ea6a24da162d634a57793cb8f9 100644 |
| --- a/Source/core/css/SelectorChecker.cpp |
| +++ b/Source/core/css/SelectorChecker.cpp |
| @@ -781,7 +781,7 @@ bool SelectorChecker::checkPseudoClass(const SelectorCheckingContext& context, c |
| case CSSSelector::PseudoHover: |
| // If we're in quirks mode, then hover should never match anchors with no |
| // href and *:hover should not match anything. This is important for sites like wsj.com. |
|
kochi
2015/01/15 10:04:11
Could you move this comment and the comment for Ps
rune
2015/01/15 10:29:02
Done.
I removed the reference to wsj.com since th
|
| - if (m_strictParsing || context.isSubSelector || element.isLink()) { |
| + if (shouldMatchHoverOrActive(context)) { |
| if (m_mode == ResolvingStyle) { |
| if (context.elementStyle) |
| context.elementStyle->setAffectedByHover(); |
| @@ -795,7 +795,7 @@ bool SelectorChecker::checkPseudoClass(const SelectorCheckingContext& context, c |
| case CSSSelector::PseudoActive: |
| // If we're in quirks mode, then :active should never match anchors with no |
| // href and *:active should not match anything. |
| - if (m_strictParsing || context.isSubSelector || element.isLink()) { |
| + if (shouldMatchHoverOrActive(context)) { |
| if (m_mode == ResolvingStyle) { |
| if (context.elementStyle) |
| context.elementStyle->setAffectedByActive(); |