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

Unified Diff: LayoutTests/fast/table/fixed-table-layout-width-change.html

Issue 264283002: Table with fixed layout behaves like auto layout when its width is set by js instead of css. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added API isTableLayoutFixed() in RenderStyle.h Created 6 years, 7 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/fixed-table-layout-width-change.html
diff --git a/LayoutTests/fast/table/fixed-table-layout-width-change.html b/LayoutTests/fast/table/fixed-table-layout-width-change.html
new file mode 100644
index 0000000000000000000000000000000000000000..b61b8cded1d7b92ce0c6803d9d606b9eb4ca8251
--- /dev/null
+++ b/LayoutTests/fast/table/fixed-table-layout-width-change.html
@@ -0,0 +1,32 @@
+<!DOCTYPE html>
+<html>
+ <head>
+ <script type="text/javascript">
+ function runTest() {
+ outer = document.getElementById('outer');
Julien - ping for review 2014/05/07 00:35:29 "var outer" to avoid attaching outer to the global
+ outer.style.width = '100px';
+ window.checkLayout("#outer", document.getElementById("test-output"));
+ };
+ </script>
+ <script src="../../resources/check-layout.js"></script>
+ <style>
+ #outer {
+ display: table;
+ table-layout: fixed;
+ }
+ #inner {
+ display: table-cell;
+ height: 50px;
+ background-color: green;
+ min-width: 200px;
+ }
+ </style>
+ </head>
+ <body onload="runTest()">
+ Tests that the width of table changes on changing the table width to new width.New width should be 100.
Julien - ping for review 2014/05/07 00:35:29 Awesome that you considered mentioning the new wid
+ <div id="outer" data-expected-width="100">
+ <div id="inner"></div>
+ </div>
+ <div id="test-output"></div>
+ </body>
+</html>

Powered by Google App Engine
This is Rietveld 408576698