Trigger repaint on first paint only on pending stylesheet decrement.

Trigger repaint on first paint only on pending stylesheet decrement.

Also, since Document::styleResolverChanged() is now just a call to
StyleEngine::resolverChanged(), remove it.

This is in preparation for splitting (style)resolverChanged() into more
descriptive methods on StyleEngine for what is necessary to nuke and
rebuild in the various cases. That, in turn, is in preparation for the
componentized style resolver and asynchronous update of active
stylesheets.

BUG=401359, 567021
Committed: https://crrev.com/56b2dd877e857cda63f913bb8b53b0037dee9477
Cr-Commit-Position: refs/heads/master@{#376594}

[rune, 2016-02-19 13:42:53]

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

[commit-bot: I haz the power, 2016-02-19 13:44:36]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1716803002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1716803002/1

[commit-bot: I haz the power, 2016-02-19 16:09:45]

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

[commit-bot: I haz the power, 2016-02-19 16:09:46]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[rune, 2016-02-19 18:33:54]

The only functional change here is that the full paint invalidation triggered
when we don't have pending styleSheets is moved to styleResolverMayHaveChanged()
(called when pending stylesheet count reaches 0) instead of checking for it for
every (style)resolverChanged().

[rune, 2016-02-19 22:53:48]

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

[esprehn, 2016-02-19 22:59:54]

What's the behavior change here? It looks like just lots of renames... I don't
see anything new.

[esprehn, 2016-02-19 23:00:38]

I see, so no behavior changes at all, we just don't do the check as often?

[rune, 2016-02-19 23:05:02]

On 2016/02/19 23:00:38, esprehn wrote:
> I see, so no behavior changes at all, we just don't do the check as often?

Right.

[esprehn, 2016-02-19 23:57:59]

The CQ bit was checked by esprehn@chromium.org

[esprehn, 2016-02-19 23:58:00]

lgtm

[commit-bot: I haz the power, 2016-02-19 23:58:50]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1716803002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1716803002/1

[commit-bot: I haz the power, 2016-02-20 01:24:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-20 01:26:52]

Description was changed from

==========
Trigger repaint on first paint only on pending stylesheet decrement.

Also, since Document::styleResolverChanged() is now just a call to
StyleEngine::resolverChanged(), remove it.

This is in preparation for splitting (style)resolverChanged() into more
descriptive methods on StyleEngine for what is necessary to nuke and
rebuild in the various cases. That, in turn, is in preparation for the
componentized style resolver and asynchronous update of active
stylesheets.

BUG=401359,567021
==========

to

==========
Trigger repaint on first paint only on pending stylesheet decrement.

Also, since Document::styleResolverChanged() is now just a call to
StyleEngine::resolverChanged(), remove it.

This is in preparation for splitting (style)resolverChanged() into more
descriptive methods on StyleEngine for what is necessary to nuke and
rebuild in the various cases. That, in turn, is in preparation for the
componentized style resolver and asynchronous update of active
stylesheets.

BUG=401359,567021

Committed: https://crrev.com/56b2dd877e857cda63f913bb8b53b0037dee9477
Cr-Commit-Position: refs/heads/master@{#376594}
==========

[commit-bot: I haz the power, 2016-02-20 01:26:54]

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