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

Unified Diff: LayoutTests/fast/table/table-rowspan-cell-with-empty-cell.html

Issue 47923009: Table rows are incorrectly collapsed in case of hidden cells and rowspans. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Review comments addressed Created 6 years, 10 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/table/table-rowspan-cell-with-empty-cell.html
diff --git a/LayoutTests/fast/table/table-rowspan-cell-with-empty-cell.html b/LayoutTests/fast/table/table-rowspan-cell-with-empty-cell.html
new file mode 100644
index 0000000000000000000000000000000000000000..ade5fe5c0a5e91b0d00fef79b74d9eecc0447a82
--- /dev/null
+++ b/LayoutTests/fast/table/table-rowspan-cell-with-empty-cell.html
@@ -0,0 +1,36 @@
+<!DOCTYPE html>
+<html>
+<head>
+ <script src="../../resources/check-layout.js"></script>
+ <style>
+ td { font: 15px/1 Ahem }
+ table {border-collapse: collapse;}
+ .dn {display: none;}
+ .blue {background: blue;}
+ .green {background: green;}
+ </style>
+</head>
+<body onload="checkLayout('#a')">
+<h3>Test for chromium bug : <a href="https://code.google.com/p/chromium/issues/detail?id=258420">258420</a>. Table rows are incorrectly collapsed in case of hidden cells and rowspans.</h3>
+<h4>A spanning cell whose rows have only empty cell(s) shouldn't have a non-zero height.</h4>
+ <table>
+ <tbody>
+ <tr class="blue">
+ <td rowspan="3">A</td>
+ <td>A1</td>
+ <td>A1.1</td>
+ </tr>
+ <tr>
+ <td rowspan="2" id="a" data-expected-width = "56">A2</td>
+ <td class="dn">A2.1</td>
+ </tr>
+ <tr>
+ <td class="dn">A2.2</td>
+ </tr>
+ <tr class="green">
+ <td colspan="3">long last row</td>
+ </tr>
+ </tbody>
+ </table>
+ </body>
+</html>
« no previous file with comments | « LayoutTests/TestExpectations ('k') | LayoutTests/fast/table/table-rowspan-cell-with-empty-cell-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698