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

Unified Diff: LayoutTests/fast/css/pseudo-not-empty-adjacent-dynamic.html

Issue 558333002: Don't clear StyleAffectedByEmpty before recalc. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 6 years, 3 months 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: LayoutTests/fast/css/pseudo-not-empty-adjacent-dynamic.html
diff --git a/LayoutTests/fast/css/pseudo-not-empty-adjacent-dynamic.html b/LayoutTests/fast/css/pseudo-not-empty-adjacent-dynamic.html
new file mode 100644
index 0000000000000000000000000000000000000000..b6671150c0e9a35606e6905b803a6301b754154f
--- /dev/null
+++ b/LayoutTests/fast/css/pseudo-not-empty-adjacent-dynamic.html
@@ -0,0 +1,24 @@
+<!DOCTYPE html>
+<script src="../../resources/js-test.js"></script>
+<style>
+#empty:not(:empty) + div { background-color: green; }
+</style>
+<div id="empty"></div>
+<div id="sibling">This text should be green</div>
+<script>
+description("Check that the sibling of an element that becomes :not(:empty) is properly recalculated");
+
+var transparent = "rgba(0, 0, 0, 0)";
+var green = "rgb(0, 128, 0)";
+
+empty.offsetTop; // force recalc
+empty.style.color = "red"; // force LocalStyleChange on #empty
+empty.offsetTop; // force recalc
+
+shouldBe("getComputedStyle(sibling, '').backgroundColor", "transparent");
+
+empty.appendChild(document.createElement("div"));
+empty.offsetTop; // force recalc
+
+shouldBe("getComputedStyle(sibling, '').backgroundColor", "green");
+</script>

Powered by Google App Engine
This is Rietveld 408576698