Revert of Remove clearing of pending invalidation sets. (patchset #2 id:20001 of https://codereview…

Revert of Remove clearing of pending invalidation sets. (patchset #2 id:20001 of https://codereview.chromium.org/1507653002/ )

Reason for revert:
Suspect root cause of crash in blink::StyleInvalidator::pushInvalidationSetsForElement

http://crbug.com/568502

Original issue's description:
> Remove clearing of pending invalidation sets.
>
> In preparation for making SubtreeStyleChange not affect the sibling
> forest.
>
> We can skip scheduling descendant invalidation sets for elements whose
> styleChangeType is SubtreeStyleChange. However, with sibling invalidation
> sets, we still need to schedule invalidations for invalidating the
> sibling forest when we change SubtreeStyleChange to be a strict subtree
> recalc.
>
> We may not clear pending invalidations on detach either, unless the node
> was actually removed from the dom tree, since there might be pending
> sibling invalidations.
>
> R=dstockwell@chromium.org,ericwilligers@chromium.org
> BUG=557440
>
> Committed: https://crrev.com/c0735df3d8f8efa24af168a6e8d5c02a7be0641c
> Cr-Commit-Position: refs/heads/master@{#364064}

TBR=dstockwell@chromium.org,ericwilligers@chromium.org,rune@opera.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557440, 568502
Review URL: https://codereview.chromium.org/1518793002

Cr-Commit-Position: refs/heads/master@{#364416}
(cherry picked from commit 91a8470f2902978d9fbdc6de3618f6f7d374c014)

Committed: https://chromium.googlesource.com/chromium/src/+/4b5d7d5fbd79a9323b05e499c98e0b206659c08a

[wkorman, 2015-12-10 19:19:41]

Description was changed from

==========
Revert of Remove clearing of pending invalidation sets. (patchset #2 id:20001 of
https://codereview.chromium.org/1507653002/ )

Reason for revert:
Suspect root cause of crash in
blink::StyleInvalidator::pushInvalidationSetsForElement

http://crbug.com/568502

Original issue's description:
> Remove clearing of pending invalidation sets.
>
> In preparation for making SubtreeStyleChange not affect the sibling
> forest.
>
> We can skip scheduling descendant invalidation sets for elements whose
> styleChangeType is SubtreeStyleChange. However, with sibling invalidation
> sets, we still need to schedule invalidations for invalidating the
> sibling forest when we change SubtreeStyleChange to be a strict subtree
> recalc.
>
> We may not clear pending invalidations on detach either, unless the node
> was actually removed from the dom tree, since there might be pending
> sibling invalidations.
>
> R=dstockwell@chromium.org,ericwilligers@chromium.org
> BUG=557440
>
> Committed: https://crrev.com/c0735df3d8f8efa24af168a6e8d5c02a7be0641c
> Cr-Commit-Position: refs/heads/master@{#364064}

TBR=dstockwell@chromium.org,ericwilligers@chromium.org,rune@opera.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557440,568502

Review URL: https://codereview.chromium.org/1518793002

Cr-Commit-Position: refs/heads/master@{#364416}
(cherry picked from commit 91a8470f2902978d9fbdc6de3618f6f7d374c014)
==========

to

==========
Revert of Remove clearing of pending invalidation sets. (patchset #2 id:20001 of
https://codereview.chromium.org/1507653002/ )

Reason for revert:
Suspect root cause of crash in
blink::StyleInvalidator::pushInvalidationSetsForElement

http://crbug.com/568502

Original issue's description:
> Remove clearing of pending invalidation sets.
>
> In preparation for making SubtreeStyleChange not affect the sibling
> forest.
>
> We can skip scheduling descendant invalidation sets for elements whose
> styleChangeType is SubtreeStyleChange. However, with sibling invalidation
> sets, we still need to schedule invalidations for invalidating the
> sibling forest when we change SubtreeStyleChange to be a strict subtree
> recalc.
>
> We may not clear pending invalidations on detach either, unless the node
> was actually removed from the dom tree, since there might be pending
> sibling invalidations.
>
> R=dstockwell@chromium.org,ericwilligers@chromium.org
> BUG=557440
>
> Committed: https://crrev.com/c0735df3d8f8efa24af168a6e8d5c02a7be0641c
> Cr-Commit-Position: refs/heads/master@{#364064}

TBR=dstockwell@chromium.org,ericwilligers@chromium.org,rune@opera.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=557440,568502

Review URL: https://codereview.chromium.org/1518793002

Cr-Commit-Position: refs/heads/master@{#364416}
(cherry picked from commit 91a8470f2902978d9fbdc6de3618f6f7d374c014)

Committed:
https://chromium.googlesource.com/chromium/src/+/4b5d7d5fbd79a9323b05e499c98e...
==========

[wkorman, 2015-12-10 19:19:43]

Committed patchset #1 (id:1) manually as
4b5d7d5fbd79a9323b05e499c98e0b206659c08a (presubmit successful).