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

Unified Diff: third_party/WebKit/LayoutTests/fragmentation/multi-line-cells-paginated.html

Issue 2382043003: Use ceil() when integerizing pagination struts before table rows. (Closed)
Patch Set: rebase master Created 4 years, 2 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/fragmentation/multi-line-cells-paginated.html
diff --git a/third_party/WebKit/LayoutTests/fragmentation/multi-line-cells-paginated.html b/third_party/WebKit/LayoutTests/fragmentation/multi-line-cells-paginated.html
index 29f1b5cff47939c354c8532ea46778d9d85e096c..4273404dcc4ff6c3b90485e18333b43589a24a28 100644
--- a/third_party/WebKit/LayoutTests/fragmentation/multi-line-cells-paginated.html
+++ b/third_party/WebKit/LayoutTests/fragmentation/multi-line-cells-paginated.html
@@ -1,12 +1,13 @@
<!DOCTYPE html>
<style>
+body { overflow: hidden; } /* Avoid triggering a second layout pass, as that might hide bugs. */
table { border-spacing:0; line-height:1.5em; }
tr { break-inside:avoid; }
td { padding:0; background:red; }
td > div { width:2em; background:green; }
</style>
<p>crbug.com/99124: A multi-line cell doesn't straddle a page-break when break-inside:avoid is specified on the row. There should be a green square and no red below.</p>
-<div style="columns:2; column-fill:auto; column-gap:0; width:4em; height:7.9em;">
+<div style="columns:2; column-fill:auto; column-gap:0; width:4em; height:8.5em;">
<table>
<tr>
<td><div><br><br></div></td>

Powered by Google App Engine
This is Rietveld 408576698