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

Unified Diff: third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution.html

Issue 2534413004: Made varied number of cells in each row based on row's requirement. (Closed)
Patch Set: Reset TestExpectations file Created 3 years, 11 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution-expected.txt » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution.html
diff --git a/third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution.html b/third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution.html
new file mode 100644
index 0000000000000000000000000000000000000000..0e2245b04054d6af04e5ec3e1ff445f9062fe273
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution.html
@@ -0,0 +1,51 @@
+<style>
+ table {
+ border-collapse: collapse;
+ }
+ td {
+ border: 1px solid #000000;
+ }
+</style>
+<script src="../../resources/check-layout.js"></script>
+<script>
+ function calls() {
+ removeCol();
+ checkLayout('tr');
+ }
+
+ function removeCol() {
+ var mutatedTable = document.getElementById("mutated-table");
+ var trElements = mutatedTable.getElementsByTagName('tr');
+
+ for (var i = 0; i < 3; i++) {
+ // Remove the third child in each row
+ var tr = trElements[i];
+ var tdElements = tr.getElementsByTagName('td');
+ var td = tdElements[2];
+ tr.removeChild(td)
+ }
+ }
+</script>
+<body onload="calls()">
+ <h3>Test for chromium bug : <a href="https://bugs.chromium.org/p/chromium/issues/detail?id=396653">396653</a>. Tables with specific merge cell configuration render improperly when removing table column.</h3>
+ <h4>Second row is rowspan-only-cell and some empty cells present in the row because td node is deleted from the dom tree using script. So Please check that second row height should not be zero in this case.</h4>
+ <table id="mutated-table" style="width: 624px;">
+ <tbody>
+ <tr data-expected-height="32">
+ <td rowspan="2" style="height: 117px;"></td>
+ <td style="height: 32px;"></td>
+ <td style="height: 32px;"></td>
+ </tr>
+ <tr data-expected-height="85">
+ <td style="height: 32px; display: none;"></td>
+ <td rowspan="2" style="height: 117px;"></td>
+ <td style="height: 84px;"></td>
+ </tr>
+ <tr data-expected-height="32">
+ <td style="height: 32px;"></td>
+ <td style="height: 32px; display: none;"></td>
+ <td style="height: 32px;"></td>
+ </tr>
+ </tbody>
+ </table>
+</body>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/fast/table/rowspan-only-rows-height-distribution-expected.txt » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698