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

Unified Diff: LayoutTests/fast/block/margin-collapse/line-beside-float-complex-margin-collapsing.html

Issue 1315353005: Avoid duplicated code in LayoutBlockChild::layoutBlockChild(). (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Resurrected comment about first layout pass and improved it. Created 5 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/block/margin-collapse/line-beside-float-complex-margin-collapsing.html
diff --git a/LayoutTests/fast/block/margin-collapse/line-beside-float-complex-margin-collapsing.html b/LayoutTests/fast/block/margin-collapse/line-beside-float-complex-margin-collapsing.html
new file mode 100644
index 0000000000000000000000000000000000000000..14c10ec8fb1a43170c053bb5a32272043e480d5c
--- /dev/null
+++ b/LayoutTests/fast/block/margin-collapse/line-beside-float-complex-margin-collapsing.html
@@ -0,0 +1,32 @@
+<!DOCTYPE html>
+<style>
+ body { margin:8px; font-size:16px; }
+</style>
+<script src="../../../resources/check-layout.js"></script>
+<p>Given a float followed by a regular block with the same height as the float, then a block B with
+ a negative margin: Check that a line inside B doesn't overlap with the float. There's an empty
+ collapse-through block before B, so that our initial logical top estimate fails and we have to
+ relayout. When laying out again, we have to detect that the float that we first thought didn't
+ affect the line now affects it.</p>
+<p>The word "BINGO" should be seen below, to the right of a blue block.</p>
+<div style="width:20em; color:blue;">
+ <div>
+ <div style="float:left; width:8em; height:8em; margin-right:1em; background:blue;"></div>
+ <div style="height:8em;"></div>
+ <div>
+ <!-- Here's an empty block that we can just collapse through, but we don't realize that
+ when calculating our initial top estimate on its parent, so we have to relayout
+ when it turns out that a negative margin has pulled us upwards (and suddenly we
+ have a line that's affected by the float). -->
+ <div></div>
+
+ <div class="testee" data-total-x="8" style="margin-top:-4em;">
+ <span class="testee" data-total-x="152">BINGO</span>
+ </div>
+ </div>
+ </div>
+</div>
+<p id="result" style="clear:both;"></p>
+<script>
+ checkLayout(".testee", document.getElementById("result"));
+</script>

Powered by Google App Engine
This is Rietveld 408576698