Remove incorrect styleResolver() checks.

Remove incorrect styleResolver() checks.

Whether the StyleResolver is created or not does no longer tell if we
have usable invalidation sets or not. Removing SubtreeStyleChange which
is presumably not necessary anymore.

Add a check for not scheduling any invalidation sets if the document is
already marked for full recalc.

Review-Url: https://codereview.chromium.org/2749273004
Cr-Commit-Position: refs/heads/master@{#457383}
Committed: https://chromium.googlesource.com/chromium/src/+/5fef4a38bd9d3121a93e206d46287794788ea05a

[rune, 2017-03-15 15:41:50]

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

[commit-bot: I haz the power, 2017-03-15 15:42:06]

Dry run: 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, 2017-03-15 17:21:07]

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

[commit-bot: I haz the power, 2017-03-15 17:21:08]

Dry run: This issue passed the CQ dry run.

[rune, 2017-03-15 22:29:16]

rune@opera.com changed reviewers:
+ meade@chromium.org

[rune, 2017-03-15 22:29:17]

ptal

[meade_UTC10, 2017-03-16 05:27:06]

lgtm

Is there a bug this was causing, or is it just an optimisation?

[rune, 2017-03-16 09:29:44]

On 2017/03/16 05:27:06, Eddy (OOO until 27 Mar) wrote:
> lgtm
> 
> Is there a bug this was causing, or is it just an optimisation?

Just an optimization.

[rune, 2017-03-16 09:29:50]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2017-03-16 09:30:02]

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, 2017-03-16 09:33:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1489656590458130, "parent_rev":
"a757a98d4a7f12ac9b53135dfd8e1452b5455920", "commit_rev":
"5fef4a38bd9d3121a93e206d46287794788ea05a"}

[commit-bot: I haz the power, 2017-03-16 09:34:30]

Description was changed from

==========
Remove incorrect styleResolver() checks.

Whether the StyleResolver is created or not does no longer tell if we
have usable invalidation sets or not. Removing SubtreeStyleChange which
is presumably not necessary anymore.

Add a check for not scheduling any invalidation sets if the document is
already marked for full recalc.
==========

to

==========
Remove incorrect styleResolver() checks.

Whether the StyleResolver is created or not does no longer tell if we
have usable invalidation sets or not. Removing SubtreeStyleChange which
is presumably not necessary anymore.

Add a check for not scheduling any invalidation sets if the document is
already marked for full recalc.

Review-Url: https://codereview.chromium.org/2749273004
Cr-Commit-Position: refs/heads/master@{#457383}
Committed:
https://chromium.googlesource.com/chromium/src/+/5fef4a38bd9d3121a93e206d4628...
==========

[commit-bot: I haz the power, 2017-03-16 09:34:32]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5fef4a38bd9d3121a93e206d4628...