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

Unified Diff: LayoutTests/fast/multicol/dynamic/relayout-inside-overflow-hidden.html

Issue 1271513004: Disallow relayout roots inside multicol. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Code review - more tests - put everything in fast/multicol/dynamic/ Created 5 years, 4 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/multicol/dynamic/relayout-inside-overflow-hidden.html
diff --git a/LayoutTests/fast/multicol/dynamic/relayout-inside-overflow-hidden.html b/LayoutTests/fast/multicol/dynamic/relayout-inside-overflow-hidden.html
new file mode 100644
index 0000000000000000000000000000000000000000..a7035143fa3cf6c2b314e146fa64489d25585e23
--- /dev/null
+++ b/LayoutTests/fast/multicol/dynamic/relayout-inside-overflow-hidden.html
@@ -0,0 +1,19 @@
+<!DOCTYPE html>
+<p>Trigger layout inside a clipped box inside a multicol container that has a spanner. Then trigger
+ layout outside the clipped box, but still inside the multicol container. This tests that we
+ don't perform any optimized subtree layout operations incompatible with multicol (which would
+ typically cause assertion failures).</p>
+<p>You should see a blue square below, and nothing else (no red, no "FAIL").</p>
+<div id="multicol" style="-webkit-columns:2; line-height:2em;">
+ <div style="width:5em; height:2em; overflow:hidden;">
+ <div id="block" style="-webkit-column-break-inside:avoid; width:3em; height:2em; background:blue;"></div>
+ <div style="-webkit-column-span:all; width:4em; height:2em; background:blue;"></div>
+ <div style="background:red;">FAIL</div>
+ </div>
+</div>
+<script>
+ document.body.offsetTop;
+ document.getElementById("block").style.width = "4em";
+ document.body.offsetTop;
+ document.getElementById("multicol").style.width = "50%";
+</script>

Powered by Google App Engine
This is Rietveld 408576698