Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(62)

Unified Diff: third_party/WebKit/LayoutTests/fast/css/modify-stylesheet-minimal-recalc-style.html

Issue 2557533005: Collect active stylesheets and and apply asynchronously. (Closed)
Patch Set: Rebased. Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/fast/css/modify-stylesheet-minimal-recalc-style.html
diff --git a/third_party/WebKit/LayoutTests/fast/css/modify-stylesheet-minimal-recalc-style.html b/third_party/WebKit/LayoutTests/fast/css/modify-stylesheet-minimal-recalc-style.html
index 40814c59175bf4f780e1183fbe7436752fbac9a4..f8b755fb0ade60afa9997f3fb217d1dc227faddf 100644
--- a/third_party/WebKit/LayoutTests/fast/css/modify-stylesheet-minimal-recalc-style.html
+++ b/third_party/WebKit/LayoutTests/fast/css/modify-stylesheet-minimal-recalc-style.html
@@ -56,8 +56,7 @@ if (window.internals) {
// Test deleteRule
forceRecalc();
sheet.sheet.deleteRule(0); // Deletes .bar rule.
- // FIXME: Ideally we'd do StyleInvalidationAnalysis here and only recalc 3 bar's, 3 baz's + documentElement
- shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "43");
+ shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "6");
shouldBe("getComputedStyle(document.querySelector('.foo')).color", '"rgb(0, 0, 0)"');
shouldBe("getComputedStyle(document.querySelector('.baz')).color", '"rgb(255, 0, 0)"');
@@ -66,8 +65,7 @@ if (window.internals) {
// Test insertRule
forceRecalc();
sheet.sheet.insertRule(".foo { color: red; }", 0);
- // FIXME: Ideally we'd do StyleInvalidationAnalysis here and only recalc 3 foo's, 3 baz's + documentElement
- shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "59");
+ shouldBe("internals.updateStyleAndReturnAffectedElementCount()", "6");
shouldBe("getComputedStyle(document.querySelector('.foo')).color", '"rgb(255, 0, 0)"');
shouldBe("getComputedStyle(document.querySelector('.baz')).color", '"rgb(255, 0, 0)"');

Powered by Google App Engine
This is Rietveld 408576698