Skip independent inherited property propagation to pseudo elements.

Skip independent inherited property propagation to pseudo elements.

UpdatePseudoElements and IndependentInherit conflict in the following
way. If we both have an independent inherit change on the actual dom
element, and we detect that we need to update the style for the pseudo
element we need to signal the inheritance propagation to the real dom
children and signal the pseudo element recalc to the pseudo element
children. If we return IndependentInherit, we lose the information
about the need for a pseudo element recalc, and if we return
UpdatePseudoElement, we lose the inheritance propagation for the actual
dom children.

We could introduce a new IndependentInheritAndUpdatePseudoElements, but
if there exists pseudo element, we would always return this constant,
so instead just force recalc on pseudo elements on IndependentInherit.

R=sashab@chromium.org
BUG=660735, 660089, 657283
Committed: https://crrev.com/998f4790884a34c5ac5167876a13b7cb7495a617
Cr-Commit-Position: refs/heads/master@{#431430}

[rune, 2016-11-10 09:43:25]

The CQ bit was checked by rune@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-10 09:43:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[rune, 2016-11-10 09:56:50]

ptal

[commit-bot: I haz the power, 2016-11-10 10:56:48]

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

[commit-bot: I haz the power, 2016-11-10 10:56:49]

Dry run: This issue passed the CQ dry run.

[sashab, 2016-11-10 23:28:56]

LGTM! :D

[sashab, 2016-11-10 23:46:53]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2016-11-10 23:47:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-11 00:51:08]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-11 00:56:34]

Description was changed from

==========
Skip independent inherited property propagation to pseudo elements.

UpdatePseudoElements and IndependentInherit conflict in the following
way. If we both have an independent inherit change on the actual dom
element, and we detect that we need to update the style for the pseudo
element we need to signal the inheritance propagation to the real dom
children and signal the pseudo element recalc to the pseudo element
children. If we return IndependentInherit, we lose the information
about the need for a pseudo element recalc, and if we return
UpdatePseudoElement, we lose the inheritance propagation for the actual
dom children.

We could introduce a new IndependentInheritAndUpdatePseudoElements, but
if there exists pseudo element, we would always return this constant,
so instead just force recalc on pseudo elements on IndependentInherit.

R=sashab@chromium.org
BUG=660735,660089,657283
==========

to

==========
Skip independent inherited property propagation to pseudo elements.

UpdatePseudoElements and IndependentInherit conflict in the following
way. If we both have an independent inherit change on the actual dom
element, and we detect that we need to update the style for the pseudo
element we need to signal the inheritance propagation to the real dom
children and signal the pseudo element recalc to the pseudo element
children. If we return IndependentInherit, we lose the information
about the need for a pseudo element recalc, and if we return
UpdatePseudoElement, we lose the inheritance propagation for the actual
dom children.

We could introduce a new IndependentInheritAndUpdatePseudoElements, but
if there exists pseudo element, we would always return this constant,
so instead just force recalc on pseudo elements on IndependentInherit.

R=sashab@chromium.org
BUG=660735,660089,657283

Committed: https://crrev.com/998f4790884a34c5ac5167876a13b7cb7495a617
Cr-Commit-Position: refs/heads/master@{#431430}
==========

[commit-bot: I haz the power, 2016-11-11 00:56:35]

Patchset 1 (id:??) landed as
https://crrev.com/998f4790884a34c5ac5167876a13b7cb7495a617
Cr-Commit-Position: refs/heads/master@{#431430}