Use hash set instead of vector for changed RuleSets.

Use hash set instead of vector for changed RuleSets.

That way, we don't have to consider the same RuleSet multiple times for
invalidation on active stylesheet update. This fixes a regression in
PerformanceTests/CSS/StyleSheetInsert.html which would have been
introduced by https://codereview.chromium.org/2557533005

This works because the same style element source text used multiple
times will make us use the same StyleSheetContents from the cache and
hence the same RuleSet for all 50 sheets added in that test. It's a bit
like cheating, but this will also make sure we don't invalidate for the
same RuleSet twice if we re-order stylesheets by removing/inserting a
style element where the CSSStyleSheet pointer will be different, but
the RuleSet stays the same.

R=sashab@chromium.org,esprehn@chromium.org
BUG=567021
Committed: https://crrev.com/5eefd373b4c293d23cb5f69a7e2fc0620eaead3b
Cr-Commit-Position: refs/heads/master@{#438062}

[rune, 2016-12-12 22:03:15]

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

[commit-bot: I haz the power, 2016-12-12 22:05:25]

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

[rune, 2016-12-12 22:05:53]

ptal

[esprehn, 2016-12-12 22:08:09]

For real world content this might be a bit slower, but it does scale better. :)

lgtm

[commit-bot: I haz the power, 2016-12-13 02:30:45]

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

[commit-bot: I haz the power, 2016-12-13 02:30:45]

Dry run: This issue passed the CQ dry run.

[rune, 2016-12-13 05:13:33]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2016-12-13 05:13:51]

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-12-13 05:18:34]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481606013967040, "parent_rev":
"ab03e07e7a9c03f6cbf5ea8fa7ab5684d548940e", "commit_rev":
"aed096709ab0996089d1536f6b5314f03e90b0a9"}

[commit-bot: I haz the power, 2016-12-13 05:19:51]

Description was changed from

==========
Use hash set instead of vector for changed RuleSets.

That way, we don't have to consider the same RuleSet multiple times for
invalidation on active stylesheet update. This fixes a regression in
PerformanceTests/CSS/StyleSheetInsert.html which would have been
introduced by https://codereview.chromium.org/2557533005

This works because the same style element source text used multiple
times will make us use the same StyleSheetContents from the cache and
hence the same RuleSet for all 50 sheets added in that test. It's a bit
like cheating, but this will also make sure we don't invalidate for the
same RuleSet twice if we re-order stylesheets by removing/inserting a
style element where the CSSStyleSheet pointer will be different, but
the RuleSet stays the same.

R=sashab@chromium.org,esprehn@chromium.org
BUG=567021
==========

to

==========
Use hash set instead of vector for changed RuleSets.

That way, we don't have to consider the same RuleSet multiple times for
invalidation on active stylesheet update. This fixes a regression in
PerformanceTests/CSS/StyleSheetInsert.html which would have been
introduced by https://codereview.chromium.org/2557533005

This works because the same style element source text used multiple
times will make us use the same StyleSheetContents from the cache and
hence the same RuleSet for all 50 sheets added in that test. It's a bit
like cheating, but this will also make sure we don't invalidate for the
same RuleSet twice if we re-order stylesheets by removing/inserting a
style element where the CSSStyleSheet pointer will be different, but
the RuleSet stays the same.

R=sashab@chromium.org,esprehn@chromium.org
BUG=567021

Review-Url: https://codereview.chromium.org/2569733003
==========

[commit-bot: I haz the power, 2016-12-13 05:19:52]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-13 05:24:52]

Description was changed from

==========
Use hash set instead of vector for changed RuleSets.

That way, we don't have to consider the same RuleSet multiple times for
invalidation on active stylesheet update. This fixes a regression in
PerformanceTests/CSS/StyleSheetInsert.html which would have been
introduced by https://codereview.chromium.org/2557533005

This works because the same style element source text used multiple
times will make us use the same StyleSheetContents from the cache and
hence the same RuleSet for all 50 sheets added in that test. It's a bit
like cheating, but this will also make sure we don't invalidate for the
same RuleSet twice if we re-order stylesheets by removing/inserting a
style element where the CSSStyleSheet pointer will be different, but
the RuleSet stays the same.

R=sashab@chromium.org,esprehn@chromium.org
BUG=567021

Review-Url: https://codereview.chromium.org/2569733003
==========

to

==========
Use hash set instead of vector for changed RuleSets.

That way, we don't have to consider the same RuleSet multiple times for
invalidation on active stylesheet update. This fixes a regression in
PerformanceTests/CSS/StyleSheetInsert.html which would have been
introduced by https://codereview.chromium.org/2557533005

This works because the same style element source text used multiple
times will make us use the same StyleSheetContents from the cache and
hence the same RuleSet for all 50 sheets added in that test. It's a bit
like cheating, but this will also make sure we don't invalidate for the
same RuleSet twice if we re-order stylesheets by removing/inserting a
style element where the CSSStyleSheet pointer will be different, but
the RuleSet stays the same.

R=sashab@chromium.org,esprehn@chromium.org
BUG=567021

Committed: https://crrev.com/5eefd373b4c293d23cb5f69a7e2fc0620eaead3b
Cr-Commit-Position: refs/heads/master@{#438062}
==========

[commit-bot: I haz the power, 2016-12-13 05:24:53]

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