Don't trigger full active style update on styleSheets access.

Don't trigger full active style update on styleSheets access.

Element.styleSheets and ShadowRoot.styleSheets need to be made up-to-
date on access. We used to do a full active style update, but re-
collecting the stylesheet list should be enough, leaving the active
style dirty flags intact.

We introduce a dirty-flag for the stylesheet list in
StyleSheetCollection to avoid repeatedly re-collecting this list while
the active style is still dirty.

This coincidentally fixes issue 722826 since we do not collect
stylesheets in import shadow trees as part of the active style update,
but is now made up-to-date on request when accessing the styleSheets
collection on shadow roots inside import documents.

This fixes the performance issue 717506.

R=meade@chromium.org
BUG=717506, 722826
Review-Url: https://codereview.chromium.org/2884993002
Cr-Commit-Position: refs/heads/master@{#473846}
Committed: https://chromium.googlesource.com/chromium/src/+/e1719ffa41c44d3434095850fee7ffa5b6e57c4f

[rune, 2017-05-16 14:18:36]

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

[commit-bot: I haz the power, 2017-05-16 14:19:37]

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-05-16 14:24:07]

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

[commit-bot: I haz the power, 2017-05-16 14:24:08]

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

[rune, 2017-05-18 10:53:04]

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

[commit-bot: I haz the power, 2017-05-18 10:53:43]

Dry run: CQ is trying da patch.

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

[rune, 2017-05-18 11:31:33]

ptal

[commit-bot: I haz the power, 2017-05-18 14:35:09]

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

[commit-bot: I haz the power, 2017-05-18 14:35:09]

Dry run: This issue passed the CQ dry run.

[meade_UTC10, 2017-05-23 04:38:53]

lgtm

[rune, 2017-05-23 08:10:32]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2017-05-23 08:10:46]

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-05-23 10:08:35]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1495527032565730,
"parent_rev": "5765835a9a242bab20810ad5d60ab504902580f5", "commit_rev":
"e1719ffa41c44d3434095850fee7ffa5b6e57c4f"}

[commit-bot: I haz the power, 2017-05-23 10:08:45]

Description was changed from

==========
Don't trigger full active style update on styleSheets access.

Element.styleSheets and ShadowRoot.styleSheets need to be made up-to-
date on access. We used to do a full active style update, but re-
collecting the stylesheet list should be enough, leaving the active
style dirty flags intact.

We introduce a dirty-flag for the stylesheet list in
StyleSheetCollection to avoid repeatedly re-collecting this list while
the active style is still dirty.

This coincidentally fixes issue 722826 since we do not collect
stylesheets in import shadow trees as part of the active style update,
but is now made up-to-date on request when accessing the styleSheets
collection on shadow roots inside import documents.

This fixes the performance issue 717506.

R=meade@chromium.org
BUG=717506,722826
==========

to

==========
Don't trigger full active style update on styleSheets access.

Element.styleSheets and ShadowRoot.styleSheets need to be made up-to-
date on access. We used to do a full active style update, but re-
collecting the stylesheet list should be enough, leaving the active
style dirty flags intact.

We introduce a dirty-flag for the stylesheet list in
StyleSheetCollection to avoid repeatedly re-collecting this list while
the active style is still dirty.

This coincidentally fixes issue 722826 since we do not collect
stylesheets in import shadow trees as part of the active style update,
but is now made up-to-date on request when accessing the styleSheets
collection on shadow roots inside import documents.

This fixes the performance issue 717506.

R=meade@chromium.org
BUG=717506,722826

Review-Url: https://codereview.chromium.org/2884993002
Cr-Commit-Position: refs/heads/master@{#473846}
Committed:
https://chromium.googlesource.com/chromium/src/+/e1719ffa41c44d3434095850fee7...
==========

[commit-bot: I haz the power, 2017-05-23 10:08:46]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/e1719ffa41c44d3434095850fee7...