Use invalidation sets for some pseudo elements.

Use invalidation sets for some pseudo elements.

Allow to use class invalidation sets for selectors containing ::before,
::after, or ::backdrop. These pseudo elements are updated correctly in
Element::recalcChildStyle for LocalStyleChange on their parents.

BUG=335247
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=169298

[chrishtr, 2014-03-11 17:33:09]

Please augment the tests to verify local style recalc happens and not subtree.
Otherwise looks good.

[esprehn, 2014-03-11 17:40:55]

Can you add a recalcStyle test with the count method on internals to verify the
number of elements that go through a recalc?

[rune, 2014-03-12 14:53:43]

On 2014/03/11 17:40:55, esprehn wrote:
> Can you add a recalcStyle test with the count method on internals to verify
the
> number of elements that go through a recalc?

Done.

[rune, 2014-03-12 14:56:06]

https://codereview.chromium.org/195093004/diff/20001/LayoutTests/fast/css/dyn...
File LayoutTests/fast/css/dynamic-class-pseudo-elements.html (right):

https://codereview.chromium.org/195093004/diff/20001/LayoutTests/fast/css/dyn...
LayoutTests/fast/css/dynamic-class-pseudo-elements.html:84:
shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "8");

This is 8 and not 9 because the ::first-letter pseudo element is created as part
of the reflow. ::first-line and ::first-letter are still subtree recalcs. That's
why they're not 3.

[chrishtr, 2014-03-12 16:31:41]

lgtm

[esprehn, 2014-03-15 05:42:07]

lgtm

[commit-bot: I haz the power, 2014-03-15 05:42:11]

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

[commit-bot: I haz the power, 2014-03-15 06:09:44]

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

[commit-bot: I haz the power, 2014-03-15 06:09:44]

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

[esprehn, 2014-03-15 07:37:53]

The CQ bit was checked by esprehn@chromium.org

[commit-bot: I haz the power, 2014-03-15 07:37:57]

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

[commit-bot: I haz the power, 2014-03-15 07:56:06]

Change committed as 169298