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

Unified Diff: third_party/WebKit/LayoutTests/fast/multicol/client-rects-rtl.html

Issue 2360913004: Support for multiple block fragments in getClientRects(). (Closed)
Patch Set: fast/overflow/overflow-height-float-not-removed-crash3.html crashed because saturated LayoutUnits caused zero height Created 4 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: third_party/WebKit/LayoutTests/fast/multicol/client-rects-rtl.html
diff --git a/third_party/WebKit/LayoutTests/fast/multicol/client-rects-rtl.html b/third_party/WebKit/LayoutTests/fast/multicol/client-rects-rtl.html
index b6fa6d83eb8d0fae1b334fd331be91bf77e61636..f2ccdac02bdc6ac8ae323789cc685c8edf005260 100644
--- a/third_party/WebKit/LayoutTests/fast/multicol/client-rects-rtl.html
+++ b/third_party/WebKit/LayoutTests/fast/multicol/client-rects-rtl.html
@@ -71,12 +71,8 @@
<br><div><br><img id="t6" style="width: 25px; height: 25px; background-color: lightblue;"></div>
</div>
-<p>
- Except here, where the blue border should be around the bigger slice of the blue square, on the left.
-</p>
-
<div class="columns">
- <div id="t7" style=" margin-top: 40px; width: 25px; height: 25px; background-color: lightblue;"></div>
+ <div id="t7" style=" margin-top: 25px; width: 25px; height: 50px; background-color: lightblue;"></div>
</div>
<script>
@@ -119,7 +115,8 @@
var block = document.getElementById("t4");
range.selectNode(block);
- placeMarkersForRange(range, 1);
+ // Skip the rectangles for the two fragments established by the DIV.
+ placeMarkersForRange(range, 2);
var slider = document.getElementById("t5");
range.selectNode(slider);

Powered by Google App Engine
This is Rietveld 408576698