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

Side by Side Diff: LayoutTests/fast/dom/HTMLTableColElement/resize-table-using-col-height-vertical-writing-mode.html

Issue 151993004: Col width is not honored when dynamically updated and it would not make table narrower (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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 unified diff | Download patch
OLDNEW
(Empty)
1 <!DOCTYPE html>
2 <html>
3 <head>
4 <style>
5 col {
6 height: 200px;
7 }
8 td {
9 background-color: #66f;
10 height: 25px;
11 }
12 </style>
13 <script type="text/javascript">
14 function changeColHeightUsingHeight() {
15 var col = document.getElementById('thecol');
16 col.style.height="100px";
17 window.checkLayout("td", document.getElementById("test-output")) ;
18 }
19 </script>
20 <script src="../../../resources/check-layout.js"></script>
21 </head>
22 <body>
23 Tests that the height of table cell changes on changing the colgroup hei ght to new height.
Julien - ping for review 2014/02/24 19:25:31 It's the logical width that you are changing, talk
Gurpreet 2014/02/25 13:12:56 Initially was checking for style()->width() instea
Julien - ping for review 2014/03/01 00:27:54 It would be nice to understand why this case was c
24 <table style="-webkit-writing-mode: vertical-lr">
25 <colgroup>
26 <col id="thecol">
27 </colgroup>
28 <tbody>
29 <tr>
30 <td id="colWidth" data-expected-height="100">Hello World</td>
31 </tr>
32 </tbody>
33 </table>
34 <button onclick="changeColHeightUsingHeight();">Click me to test manuall y. The column should shrink to 100px in height.</button>
35 <div id="test-output"></div>
36 </body>
37 <script>
38 if (window.eventSender) {
39 eventSender.mouseMoveTo(20, 240);
40 eventSender.mouseDown();
41 eventSender.mouseUp();
42 }
43 </script>
44 </html>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698