Support invalidation sets for :host pseudo.

Support invalidation sets for :host pseudo.

Traverse the selectorList() for :host pseudo, extract features, and create
class invalidation sets like we do for features and classes on the top level.

R=esprehn,ojan
BUG=335247, 339729
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169927

[rune, 2014-03-24 11:26:33]

A rebased solution with review issues from [1] addressed.

[1] https://codereview.chromium.org/208013002/

[rune, 2014-03-24 12:16:45]

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

https://codereview.chromium.org/209333005/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.h:148: void addFeaturesToInvalidationSets(const
CSSSelector&, const Vector<AtomicString>& classes, AtomicString id, AtomicString
tagName, const Vector<AtomicString>& attributes);

Should consider moving these features into a struct to avoid the extensive
parameter list.

[esprehn, 2014-03-24 15:59:51]

You didn't need to close and open a new issue. :)

[esprehn, 2014-03-24 16:09:11]

lgtm, fix nits before landing.

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

https://codereview.chromium.org/209333005/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:189: return AddFeatures;
Since you're always going to return AddFeatures this should be

if (current)
  addFeaturesToInvalidationSets(*current, classes, id, tagName, attributes);
return AddFeatures;

[rune, 2014-03-24 17:17:18]

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

https://codereview.chromium.org/209333005/diff/1/Source/core/css/RuleFeature....
Source/core/css/RuleFeature.cpp:189: return AddFeatures;
On 2014/03/24 16:09:11, esprehn wrote:
> Since you're always going to return AddFeatures this should be
> 
> if (current)
>   addFeaturesToInvalidationSets(*current, classes, id, tagName, attributes);
> return AddFeatures;

Done.

[rune, 2014-03-24 17:17:23]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-03-24 17:17:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/209333005/20001

[commit-bot: I haz the power, 2014-03-24 17:44:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-24 17:44:48]

Try jobs failed on following builders:
  tryserver.blink on linux_blink_rel

[rune, 2014-03-24 21:21:22]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-03-24 21:21:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/209333005/20001

[commit-bot: I haz the power, 2014-03-24 21:22:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-24 21:22:49]

Try jobs failed on following builders:
  tryserver.blink on mac_blink_rel

[abarth-chromium, 2014-03-24 21:43:19]

This CL saves 54.5% of the time in the main style recalc in toggle_drawer (i.e.,
2.2x faster).  That reduces the latency for opening the drawer by 10ms.

[esprehn, 2014-03-24 22:35:07]

All the bubbles are red with build failure?

[rune, 2014-03-25 08:14:49]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-03-25 08:14:56]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/209333005/20001

[commit-bot: I haz the power, 2014-03-25 08:56:16]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-03-25 08:56:17]

Try jobs failed on following builders:
  tryserver.blink on win_blink_rel

[rune, 2014-03-25 09:02:57]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-03-25 09:03:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/rune@opera.com/209333005/20001

[commit-bot: I haz the power, 2014-03-25 09:36:39]

Change committed as 169927