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

Side by Side Diff: LayoutTests/fast/table/large-shrink-wrapped-width.html

Issue 988443003: Don't add artifical 1 pixel width to empty tables (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Updated Created 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 <style> 1 <style>
2 .container { 2 .container {
3 width: 2000000px; 3 width: 2000000px;
4 } 4 }
5 .child { 5 .child {
6 display:inline-block; 6 display:inline-block;
7 background-color: orange; 7 background-color: orange;
8 height: 10px; 8 height: 10px;
9 } 9 }
10 td { 10 td {
11 padding: 0; 11 padding: 0;
12 } 12 }
13 table { 13 table {
14 border-spacing: 0; 14 border-spacing: 0;
15 display: inline-table; 15 display: inline-table;
16 outline: 5px solid salmon; 16 outline: 5px solid salmon;
17 } 17 }
18 </style> 18 </style>
19 19
20 <!-- The inner div should fill the container. It doesn't right now because we ar tificially limit the table's maxPreferredLogicalWidth to 1000000. 20 <!-- The inner div should fill the container. It doesn't right now because we ar tificially limit the table's maxPreferredLogicalWidth to 1000000.
21 See the related FIXME in TableLayoutAlgorithm.h. 21 See the related FIXME in TableLayoutAlgorithm.h.
22 --> 22 -->
23 <div class="container"> 23 <div class="container">
24 <div style="display:inline-block; border: 5px solid salmon;" data-expected-w idth=2000000> 24 <div style="display:inline-block; border: 5px solid salmon;" data-expected-w idth=2000000>
25 <table style="width:100%; background-color:green; table-layout:fixed"><t r><td>Content</td></tr></table> 25 <table style="width:100%; background-color:green; table-layout:fixed"><t r><td>Content</td></tr></table>
26 </div> 26 </div>
27 </div> 27 </div>
28 28
29 <div class="container"> 29 <div class="container">
30 <!-- The 1 px extra is from the align=right td. --> 30 <!-- The 1 px extra is from the align=right td. -->
mstensho (USE GERRIT) 2015/05/18 08:45:38 Strange how align=right could do that, but now the
31 <table data-expected-width=20001> 31 <table data-expected-width=20000>
32 <td align=right></td> 32 <td align=right></td>
33 <td width="100%"> 33 <td width="100%">
34 <div class="child" style="width: 10000px"></div><div class="child" s tyle="width: 10000px"></div> 34 <div class="child" style="width: 10000px"></div><div class="child" s tyle="width: 10000px"></div>
35 </td> 35 </td>
36 </table> 36 </table>
37 </div> 37 </div>
38 38
39 <div class="container"> 39 <div class="container">
40 <!-- The 1 px extra is from the align=right td. --> 40 <!-- The 1 px extra is from the align=right td. -->
41 <table data-expected-width=1000001> 41 <table data-expected-width=1000000>
42 <td align=right></td> 42 <td align=right></td>
43 <td width="100%"> 43 <td width="100%">
44 <div class="child" style="width: 500000px"></div><div class="child" style="width: 500000px"></div> 44 <div class="child" style="width: 500000px"></div><div class="child" style="width: 500000px"></div>
45 </td> 45 </td>
46 </table> 46 </table>
47 </div> 47 </div>
48 48
49 <div class="container"> 49 <div class="container">
50 <!-- The 1 px extra is from the align=right td. --> 50 <!-- The 1 px extra is from the align=right td. -->
51 <table data-expected-width=1500001> 51 <table data-expected-width=1500000>
52 <td align=right></td> 52 <td align=right></td>
53 <td width="100%"> 53 <td width="100%">
54 <div class="child" style="width: 500000px"></div><div class="child" style="width: 500000px"></div><div class="child" style="width: 500000px"></div> 54 <div class="child" style="width: 500000px"></div><div class="child" style="width: 500000px"></div><div class="child" style="width: 500000px"></div>
55 </td> 55 </td>
56 </table> 56 </table>
57 </div> 57 </div>
58 58
59 <script src="../../resources/check-layout.js"></script> 59 <script src="../../resources/check-layout.js"></script>
60 <script> 60 <script>
61 checkLayout('.container'); 61 checkLayout('.container');
62 </script> 62 </script>
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698