Use style invalidation sets for :-webkit-autofill.

Use style invalidation sets for :-webkit-autofill.

Add invalidation features to a set for :-webkit-autofill and move
CSSSelector::PseudoAutofill away from the list of pseudo classes requiring
subtree style recalc.

The pseudo type cases in invalidationSetForSelector were re-ordered to
match the enum order.

R=chrishtr@chromium.org,esprehn@chromium.org
BUG=424581
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=183955

[chrishtr, 2014-10-17 18:47:29]

https://codereview.chromium.org/660273002/diff/1/Source/core/css/RuleFeature.cpp
File Source/core/css/RuleFeature.cpp (right):

https://codereview.chromium.org/660273002/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:303: case CSSSelector::PseudoActive:
Why did you make these not alphabetical?

[rune, 2014-10-19 20:11:02]

https://codereview.chromium.org/660273002/diff/1/Source/core/css/RuleFeature.cpp
File Source/core/css/RuleFeature.cpp (right):

https://codereview.chromium.org/660273002/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:303: case CSSSelector::PseudoActive:
On 2014/10/17 at 18:47:28, chrishtr wrote:
> Why did you make these not alphabetical?

Because I didn't know/think that was the style. I couldn't find anything about
it in the style guide. Looking at the blink code, it doesn't seem to be a common
way to order them. I can just append PseudoAutofill at the end if you think
that's better?

At least I thought it was a good idea to use the enum order in
supportsInvalidation and requiresSubtreeInvalidation to make it easy to check
against the full list in CSSSelector.h.

[chrishtr, 2014-10-19 22:55:54]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2014-10-19 22:55:55]

lgtm

https://codereview.chromium.org/660273002/diff/1/Source/core/css/RuleFeature.cpp
File Source/core/css/RuleFeature.cpp (right):

https://codereview.chromium.org/660273002/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:303: case CSSSelector::PseudoActive:
On 2014/10/19 20:11:02, rune wrote:
> On 2014/10/17 at 18:47:28, chrishtr wrote:
> > Why did you make these not alphabetical?
> 
> Because I didn't know/think that was the style. I couldn't find anything about
> it in the style guide. Looking at the blink code, it doesn't seem to be a
common
> way to order them. I can just append PseudoAutofill at the end if you think
> that's better?
> 
> At least I thought it was a good idea to use the enum order in
> supportsInvalidation and requiresSubtreeInvalidation to make it easy to check
> against the full list in CSSSelector.h.

Ok.

[commit-bot: I haz the power, 2014-10-19 22:56:42]

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

[commit-bot: I haz the power, 2014-10-20 00:01:37]

Committed patchset #1 (id:1) as 183955