Support :hover/:active in quirks when leftmost in compound.

Support :hover/:active in quirks when leftmost in compound.

SelectorChecker checked if there was a simple selector to the left of
:hover/:active. Added check for simple selector to the right too.

The relevant spec is [1].

[1] https://quirks.spec.whatwg.org/#the-:active-and-:hover-quirk

BUG=448325, 448049
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=188465

[rune, 2015-01-13 12:37:15]

rune@opera.com changed reviewers:
+ kochi@chromium.org

[kochi, 2015-01-15 10:04:11]

LGTM

https://codereview.chromium.org/848853004/diff/1/Source/core/css/SelectorChec...
File Source/core/css/SelectorChecker.cpp (right):

https://codereview.chromium.org/848853004/diff/1/Source/core/css/SelectorChec...
Source/core/css/SelectorChecker.cpp:783: // href and *:hover should not match
anything. This is important for sites like wsj.com.
Could you move this comment and the comment for PseudoActive (line796)
into header and update with the link for quirks spec?

[rune, 2015-01-15 10:29:03]

https://codereview.chromium.org/848853004/diff/1/Source/core/css/SelectorChec...
File Source/core/css/SelectorChecker.cpp (right):

https://codereview.chromium.org/848853004/diff/1/Source/core/css/SelectorChec...
Source/core/css/SelectorChecker.cpp:783: // href and *:hover should not match
anything. This is important for sites like wsj.com.
On 2015/01/15 10:04:11, Takayoshi Kochi wrote:
> Could you move this comment and the comment for PseudoActive (line796)
> into header and update with the link for quirks spec?

Done.

I removed the reference to wsj.com since that site is using strict mode now.

[rune, 2015-01-15 10:29:10]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2015-01-15 10:29:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/848853004/20001

[kochi, 2015-01-15 10:30:39]

On 2015/01/15 10:29:03, rune wrote:
>
https://codereview.chromium.org/848853004/diff/1/Source/core/css/SelectorChec...
> File Source/core/css/SelectorChecker.cpp (right):
> 
>
https://codereview.chromium.org/848853004/diff/1/Source/core/css/SelectorChec...
> Source/core/css/SelectorChecker.cpp:783: // href and *:hover should not match
> anything. This is important for sites like http://wsj.com.
> On 2015/01/15 10:04:11, Takayoshi Kochi wrote:
> > Could you move this comment and the comment for PseudoActive (line796)
> > into header and update with the link for quirks spec?
> 
> Done.
> 
> I removed the reference to http://wsj.com since that site is using strict mode
now.

Thanks, LGTM.

[commit-bot: I haz the power, 2015-01-15 11:57:22]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=188465