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

Unified Diff: third_party/WebKit/LayoutTests/fast/table/layout-section-when-table-height-changes.html

Issue 2701163002: Avoid pathological layout on nested percent height tables (Closed)
Patch Set: Created 3 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: third_party/WebKit/LayoutTests/fast/table/layout-section-when-table-height-changes.html
diff --git a/third_party/WebKit/LayoutTests/fast/table/layout-section-when-table-height-changes.html b/third_party/WebKit/LayoutTests/fast/table/layout-section-when-table-height-changes.html
new file mode 100644
index 0000000000000000000000000000000000000000..a204af9d5967cda0827308c498d74f5f152a3cbc
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/fast/table/layout-section-when-table-height-changes.html
@@ -0,0 +1,28 @@
+<!DOCTYPE html>
+<style>
+table {
+ table-layout: fixed;
+ height: 100%;
+ width: 100%;
+}
+</style>
+<p> crbug.com/691968: There should be no scrollbar. </p>
+<div id="container" style="width: 400px; height: 400px; overflow: auto; overflow-x: hidden;">
+ <table id="table" data-expected-height=200>
+ <colgroup>
+ <col style="width: 1161px;">
mstensho (USE GERRIT) 2017/02/20 13:01:31 Is the insane width necessary? Are the COLGROUP an
rhogan 2017/02/20 18:38:16 Yes, otherwise m_columnLogicalWidthChanged gets se
mstensho (USE GERRIT) 2017/02/20 19:02:32 OK, thanks for explaining. I think I used border-s
+ </colgroup>
+ <tbody>
+ <tr>
+ <td></td>
+ </tr>
+ </tbody>
+ </table>
+</div>
+<div id="results"></div>
+<script src="../../resources/check-layout.js"></script>
mstensho (USE GERRIT) 2017/02/20 13:01:31 Time to switch to check-layout-th.js , to avoid th
+<script>
+document.body.offsetTop;
+document.getElementById("container").style.height = "200px";
+checkLayout('#table', results);
+</script>

Powered by Google App Engine
This is Rietveld 408576698