Changeset 128906 in webkit


Ignore:
Timestamp:
Sep 18, 2012 10:51:22 AM (12 years ago)
Author:
jchaffraix@webkit.org
Message:

Tables without any descendant and auto logical width should have a 0px logical width
https://bugs.webkit.org/show_bug.cgi?id=95521

Reviewed by Abhishek Arya.

Source/WebCore:

The code would wrongly add the border-spacing in the row direction to the table's logical
width even if we didn't have a column. The new behavior matches FireFox and Opera. IE
matches our old behavior for inline tables but our new behavior for normal tables which
is a bug on their side.

Tests: fast/table/empty-table-should-take-no-space.html

fast/table/fixed-table-layout/empty-table-should-take-no-space-fixed-layout.html

  • rendering/RenderTable.h:

(WebCore::RenderTable::borderSpacingInRowDirection):
Added this new helper function to return the right border-spacing. Added a FIXME as the code always
return the horizontal dimension which is wrong in vertical-writing mode.

(WebCore::RenderTable::bordersPaddingAndSpacingInRowDirection):
Changed to call borderSpacingInRowDirection. Added a comment as to why we don't add border-spacing on
border-collapse: separate tables.

LayoutTests:

  • fast/table/empty-table-should-take-no-space-expected.html: Added.
  • fast/table/empty-table-should-take-no-space.html: Added.
  • fast/table/fixed-table-layout/empty-table-should-take-no-space-fixed-layout-expected.html: Added.
  • fast/table/fixed-table-layout/empty-table-should-take-no-space-fixed-layout.html: Added.

2 new tests to check that empty tables have 0px logical width.

  • platform/chromium-linux/tables/mozilla/bugs/bug56563-expected.png:
  • platform/chromium-win/tables/mozilla/bugs/bug56563-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug56563-expected.txt:
  • platform/qt/tables/mozilla/bugs/bug56563-expected.txt:
  • tables/mozilla/bugs/bug56563-expected.txt:

Progression. The table now is 2px wide as it has a 1px right & left border.

  • platform/chromium-linux/fast/invalid/residual-style-expected.txt:

Progression. The changes are due to the tables shrinking which is expected.

  • platform/chromium-linux/fast/forms/file/file-input-disabled-expected.txt:
  • platform/chromium-mac/fast/forms/file/file-input-disabled-expected.txt:
  • platform/chromium-mac/tables/mozilla/bugs/bug113235-2-expected.txt:
  • platform/chromium-win-xp/fast/forms/file-input-disabled-expected.txt:
  • platform/chromium-win/fast/forms/file/file-input-disabled-expected.txt:
  • platform/chromium-win/fast/invalid/017-expected.txt:
  • platform/chromium-win/fast/invalid/018-expected.txt:
  • platform/chromium-win/fast/invalid/020-expected.txt:
  • platform/chromium-win/fast/invalid/table-inside-stray-table-content-expected.txt:
  • platform/chromium-win/tables/mozilla/bugs/bug113235-2-expected.txt:
  • platform/chromium-win/tables/mozilla/bugs/bug23994-expected.txt:
  • platform/chromium-win/tables/mozilla/bugs/bug56405-expected.txt:
  • platform/efl/fast/forms/file/file-input-disabled-expected.txt:
  • platform/efl/fast/invalid/017-expected.txt:
  • platform/efl/fast/invalid/018-expected.txt:
  • platform/efl/fast/invalid/020-expected.txt:
  • platform/efl/fast/invalid/table-inside-stray-table-content-expected.txt:
  • platform/efl/fast/invalid/table-residual-style-crash-expected.txt:
  • platform/efl/tables/mozilla/bugs/bug23994-expected.txt:
  • platform/efl/tables/mozilla/bugs/bug56405-expected.txt:
  • platform/gtk/fast/forms/file/file-input-disabled-expected.txt:
  • platform/gtk/fast/invalid/017-expected.txt:
  • platform/gtk/fast/invalid/018-expected.txt:
  • platform/gtk/fast/invalid/020-expected.txt:
  • platform/gtk/fast/invalid/table-inside-stray-table-content-expected.txt:
  • platform/gtk/fast/invalid/table-residual-style-crash-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug23994-expected.txt:
  • platform/gtk/tables/mozilla/bugs/bug56405-expected.txt:
  • platform/mac/fast/forms/file/file-input-disabled-expected.txt:
  • platform/mac/fast/invalid/017-expected.txt:
  • platform/mac/fast/invalid/018-expected.txt:
  • platform/mac/fast/invalid/020-expected.txt:
  • platform/mac/fast/invalid/table-inside-stray-table-content-expected.txt:
  • platform/mac/fast/invalid/table-residual-style-crash-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug113235-2-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug23994-expected.txt:
  • platform/mac/tables/mozilla/bugs/bug56405-expected.txt:
  • platform/qt/fast/forms/file/file-input-disabled-expected.txt:
  • platform/qt/fast/invalid/017-expected.txt:
  • platform/qt/fast/invalid/018-expected.txt:
  • platform/qt/fast/invalid/020-expected.txt:
  • platform/qt/fast/invalid/table-inside-stray-table-content-expected.txt:
  • platform/qt/fast/invalid/table-residual-style-crash-expected.txt:
  • platform/qt/tables/mozilla/bugs/bug113235-2-expected.txt:
  • platform/qt/tables/mozilla/bugs/bug23994-expected.txt:
  • platform/qt/tables/mozilla/bugs/bug56405-expected.txt:
  • platform/win/fast/forms/file-input-disabled-expected.txt:
  • platform/win/fast/forms/file/file-input-disabled-expected.txt:
  • tables/mozilla/bugs/bug113235-2-expected.txt:

Progression. The tables now have a 0px logical width. In some cases, we add the border-spacing in the
block-flow direction if we have a row. This is not consistently handled by browsers and not very well
defined in CSS.

Location:
trunk
Files:
4 added
58 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r128904 r128906  
     12012-09-18  Julien Chaffraix  <jchaffraix@webkit.org>
     2
     3        Tables without any descendant and auto logical width should have a 0px logical width
     4        https://bugs.webkit.org/show_bug.cgi?id=95521
     5
     6        Reviewed by Abhishek Arya.
     7
     8        * fast/table/empty-table-should-take-no-space-expected.html: Added.
     9        * fast/table/empty-table-should-take-no-space.html: Added.
     10        * fast/table/fixed-table-layout/empty-table-should-take-no-space-fixed-layout-expected.html: Added.
     11        * fast/table/fixed-table-layout/empty-table-should-take-no-space-fixed-layout.html: Added.
     12        2 new tests to check that empty tables have 0px logical width.
     13
     14        * platform/chromium-linux/tables/mozilla/bugs/bug56563-expected.png:
     15        * platform/chromium-win/tables/mozilla/bugs/bug56563-expected.txt:
     16        * platform/mac/tables/mozilla/bugs/bug56563-expected.txt:
     17        * platform/qt/tables/mozilla/bugs/bug56563-expected.txt:
     18        * tables/mozilla/bugs/bug56563-expected.txt:
     19        Progression. The table now is 2px wide as it has a 1px right & left border.
     20
     21        * platform/chromium-linux/fast/invalid/residual-style-expected.txt:
     22        Progression. The changes are due to the tables shrinking which is expected.
     23
     24        * platform/chromium-linux/fast/forms/file/file-input-disabled-expected.txt:
     25        * platform/chromium-mac/fast/forms/file/file-input-disabled-expected.txt:
     26        * platform/chromium-mac/tables/mozilla/bugs/bug113235-2-expected.txt:
     27        * platform/chromium-win-xp/fast/forms/file-input-disabled-expected.txt:
     28        * platform/chromium-win/fast/forms/file/file-input-disabled-expected.txt:
     29        * platform/chromium-win/fast/invalid/017-expected.txt:
     30        * platform/chromium-win/fast/invalid/018-expected.txt:
     31        * platform/chromium-win/fast/invalid/020-expected.txt:
     32        * platform/chromium-win/fast/invalid/table-inside-stray-table-content-expected.txt:
     33        * platform/chromium-win/tables/mozilla/bugs/bug113235-2-expected.txt:
     34        * platform/chromium-win/tables/mozilla/bugs/bug23994-expected.txt:
     35        * platform/chromium-win/tables/mozilla/bugs/bug56405-expected.txt:
     36        * platform/efl/fast/forms/file/file-input-disabled-expected.txt:
     37        * platform/efl/fast/invalid/017-expected.txt:
     38        * platform/efl/fast/invalid/018-expected.txt:
     39        * platform/efl/fast/invalid/020-expected.txt:
     40        * platform/efl/fast/invalid/table-inside-stray-table-content-expected.txt:
     41        * platform/efl/fast/invalid/table-residual-style-crash-expected.txt:
     42        * platform/efl/tables/mozilla/bugs/bug23994-expected.txt:
     43        * platform/efl/tables/mozilla/bugs/bug56405-expected.txt:
     44        * platform/gtk/fast/forms/file/file-input-disabled-expected.txt:
     45        * platform/gtk/fast/invalid/017-expected.txt:
     46        * platform/gtk/fast/invalid/018-expected.txt:
     47        * platform/gtk/fast/invalid/020-expected.txt:
     48        * platform/gtk/fast/invalid/table-inside-stray-table-content-expected.txt:
     49        * platform/gtk/fast/invalid/table-residual-style-crash-expected.txt:
     50        * platform/gtk/tables/mozilla/bugs/bug23994-expected.txt:
     51        * platform/gtk/tables/mozilla/bugs/bug56405-expected.txt:
     52        * platform/mac/fast/forms/file/file-input-disabled-expected.txt:
     53        * platform/mac/fast/invalid/017-expected.txt:
     54        * platform/mac/fast/invalid/018-expected.txt:
     55        * platform/mac/fast/invalid/020-expected.txt:
     56        * platform/mac/fast/invalid/table-inside-stray-table-content-expected.txt:
     57        * platform/mac/fast/invalid/table-residual-style-crash-expected.txt:
     58        * platform/mac/tables/mozilla/bugs/bug113235-2-expected.txt:
     59        * platform/mac/tables/mozilla/bugs/bug23994-expected.txt:
     60        * platform/mac/tables/mozilla/bugs/bug56405-expected.txt:
     61        * platform/qt/fast/forms/file/file-input-disabled-expected.txt:
     62        * platform/qt/fast/invalid/017-expected.txt:
     63        * platform/qt/fast/invalid/018-expected.txt:
     64        * platform/qt/fast/invalid/020-expected.txt:
     65        * platform/qt/fast/invalid/table-inside-stray-table-content-expected.txt:
     66        * platform/qt/fast/invalid/table-residual-style-crash-expected.txt:
     67        * platform/qt/tables/mozilla/bugs/bug113235-2-expected.txt:
     68        * platform/qt/tables/mozilla/bugs/bug23994-expected.txt:
     69        * platform/qt/tables/mozilla/bugs/bug56405-expected.txt:
     70        * platform/win/fast/forms/file-input-disabled-expected.txt:
     71        * platform/win/fast/forms/file/file-input-disabled-expected.txt:
     72        * tables/mozilla/bugs/bug113235-2-expected.txt:
     73        Progression. The tables now have a 0px logical width. In some cases, we add the border-spacing in the
     74        block-flow direction if we have a row. This is not consistently handled by browsers and not very well
     75        defined in CSS.
     76
    1772012-09-18  Zan Dobersek  <zandobersek@gmail.com>
    278
  • trunk/LayoutTests/platform/chromium-linux/fast/forms/file/file-input-disabled-expected.txt

    r102411 r128906  
    2020                  text run at (0,0) width 69: "Choose File"
    2121          RenderBR {BR} at (324,59) size 0x0
    22         RenderTable {TABLE} at (0,67) size 2x0
     22        RenderTable {TABLE} at (0,67) size 0x0
  • trunk/LayoutTests/platform/chromium-linux/fast/invalid/residual-style-expected.txt

    r117821 r128906  
    142142        RenderText {#text} at (0,0) size 76x19
    143143          text run at (0,0) width 76: "CAPTION: "
    144         RenderInline {FONT} at (0,0) size 162x19 [color=#008000]
    145           RenderTable {TABLE} at (76,15) size 2x0
    146             RenderBlock {CAPTION} at (0,0) size 2x0
     144        RenderInline {FONT} at (0,0) size 160x19 [color=#008000]
     145          RenderTable {TABLE} at (76,15) size 0x0
     146            RenderBlock {CAPTION} at (0,0) size 0x0
    147147              RenderInline {FONT} at (0,0) size 0x0 [color=#FF0000]
    148           RenderText {#text} at (78,0) size 160x19
    149             text run at (78,0) width 160: "All of this should be green."
     148          RenderText {#text} at (76,0) size 160x19
     149            text run at (76,0) width 160: "All of this should be green."
    150150        RenderText {#text} at (0,0) size 0x0
    151151      RenderBlock {HR} at (0,641) size 769x3 [border: (1px inset #000000)]
     
    183183        RenderText {#text} at (0,0) size 39x19
    184184          text run at (0,0) width 39: "COL: "
    185         RenderInline {FONT} at (0,0) size 162x19 [color=#008000]
     185        RenderInline {FONT} at (0,0) size 160x19 [color=#008000]
    186186          RenderInline {FONT} at (0,0) size 0x19 [color=#FF0000]
    187           RenderTable {TABLE} at (39,15) size 2x0
     187          RenderTable {TABLE} at (39,15) size 0x0
    188188            RenderTableCol {COLGROUP} at (0,0) size 0x0
    189189              RenderTableCol {COL} at (0,0) size 0x0
    190190          RenderInline {FONT} at (0,0) size 160x19 [color=#FF0000]
    191             RenderText {#text} at (41,0) size 160x19
    192               text run at (41,0) width 160: "All of this should be green."
     191            RenderText {#text} at (39,0) size 160x19
     192              text run at (39,0) width 160: "All of this should be green."
    193193        RenderText {#text} at (0,0) size 0x0
    194194      RenderBlock {HR} at (0,793) size 769x3 [border: (1px inset #000000)]
     
    196196        RenderText {#text} at (0,0) size 92x19
    197197          text run at (0,0) width 92: "COLGROUP: "
    198         RenderInline {FONT} at (0,0) size 162x19 [color=#008000]
     198        RenderInline {FONT} at (0,0) size 160x19 [color=#008000]
    199199          RenderInline {FONT} at (0,0) size 0x19 [color=#FF0000]
    200           RenderTable {TABLE} at (92,15) size 2x0
     200          RenderTable {TABLE} at (92,15) size 0x0
    201201            RenderTableCol {COLGROUP} at (0,0) size 0x0
    202202          RenderInline {FONT} at (0,0) size 160x19 [color=#FF0000]
    203             RenderText {#text} at (94,0) size 160x19
    204               text run at (94,0) width 160: "All of this should be green."
     203            RenderText {#text} at (92,0) size 160x19
     204              text run at (92,0) width 160: "All of this should be green."
    205205        RenderText {#text} at (0,0) size 0x0
    206206      RenderBlock {HR} at (0,831) size 769x3 [border: (1px inset #000000)]
     
    720720        RenderText {#text} at (0,0) size 55x19
    721721          text run at (0,0) width 55: "TABLE: "
    722         RenderInline {FONT} at (0,0) size 162x19 [color=#008000]
     722        RenderInline {FONT} at (0,0) size 160x19 [color=#008000]
    723723          RenderInline {FONT} at (0,0) size 0x19 [color=#FF0000]
    724           RenderTable {TABLE} at (55,15) size 2x0
     724          RenderTable {TABLE} at (55,15) size 0x0
    725725          RenderInline {FONT} at (0,0) size 160x19 [color=#FF0000]
    726             RenderText {#text} at (57,0) size 160x19
    727               text run at (57,0) width 160: "All of this should be green."
     726            RenderText {#text} at (55,0) size 160x19
     727              text run at (55,0) width 160: "All of this should be green."
    728728          RenderText {#text} at (0,0) size 0x0
    729729      RenderBlock (anonymous) at (0,3223) size 769x3 [color=#008000]
    730730        RenderBlock {HR} at (0,0) size 769x2 [border: (1px inset #008000)]
    731731      RenderBlock (anonymous) at (0,3233) size 769x21
    732         RenderInline {FONT} at (0,0) size 222x19 [color=#008000]
     732        RenderInline {FONT} at (0,0) size 220x19 [color=#008000]
    733733          RenderText {#text} at (0,0) size 60x19
    734734            text run at (0,0) width 60: "TBODY: "
    735           RenderInline {FONT} at (0,0) size 162x19
     735          RenderInline {FONT} at (0,0) size 160x19
    736736            RenderInline {FONT} at (0,0) size 0x19
    737             RenderTable {TABLE} at (60,13) size 2x2
    738               RenderTableSection {TBODY} at (0,0) size 2x2
     737            RenderTable {TABLE} at (60,13) size 0x2
     738              RenderTableSection {TBODY} at (0,0) size 0x2
    739739            RenderInline {FONT} at (0,0) size 160x19
    740               RenderText {#text} at (62,0) size 160x19
    741                 text run at (62,0) width 160: "All of this should be green."
     740              RenderText {#text} at (60,0) size 160x19
     741                text run at (60,0) width 160: "All of this should be green."
    742742            RenderText {#text} at (0,0) size 0x0
    743743      RenderBlock (anonymous) at (0,3261) size 769x3 [color=#008000]
     
    760760        RenderBlock {HR} at (0,0) size 769x2 [border: (1px inset #008000)]
    761761      RenderBlock (anonymous) at (0,3309) size 769x21
    762         RenderInline {FONT} at (0,0) size 220x19 [color=#008000]
    763           RenderInline {FONT} at (0,0) size 220x19
     762        RenderInline {FONT} at (0,0) size 218x19 [color=#008000]
     763          RenderInline {FONT} at (0,0) size 218x19
    764764            RenderText {#text} at (0,0) size 58x19
    765765              text run at (0,0) width 58: "TFOOT: "
    766             RenderInline {FONT} at (0,0) size 162x19
     766            RenderInline {FONT} at (0,0) size 160x19
    767767              RenderInline {FONT} at (0,0) size 0x19
    768               RenderTable {TABLE} at (58,13) size 2x2
    769                 RenderTableSection {TFOOT} at (0,0) size 2x2
     768              RenderTable {TABLE} at (58,13) size 0x2
     769                RenderTableSection {TFOOT} at (0,0) size 0x2
    770770              RenderInline {FONT} at (0,0) size 160x19
    771                 RenderText {#text} at (60,0) size 160x19
    772                   text run at (60,0) width 160: "All of this should be green."
     771                RenderText {#text} at (58,0) size 160x19
     772                  text run at (58,0) width 160: "All of this should be green."
    773773              RenderText {#text} at (0,0) size 0x0
    774774      RenderBlock (anonymous) at (0,3337) size 769x3 [color=#008000]
    775775        RenderBlock {HR} at (0,0) size 769x2 [border: (1px inset #008000)]
    776776      RenderBlock (anonymous) at (0,3347) size 769x21
    777         RenderInline {FONT} at (0,0) size 220x19 [color=#008000]
    778           RenderInline {FONT} at (0,0) size 220x19
    779             RenderInline {FONT} at (0,0) size 220x19
     777        RenderInline {FONT} at (0,0) size 218x19 [color=#008000]
     778          RenderInline {FONT} at (0,0) size 218x19
     779            RenderInline {FONT} at (0,0) size 218x19
    780780              RenderText {#text} at (0,0) size 58x19
    781781                text run at (0,0) width 58: "THEAD: "
    782               RenderInline {FONT} at (0,0) size 162x19
     782              RenderInline {FONT} at (0,0) size 160x19
    783783                RenderInline {FONT} at (0,0) size 0x19
    784                 RenderTable {TABLE} at (58,13) size 2x2
    785                   RenderTableSection {THEAD} at (0,0) size 2x2
     784                RenderTable {TABLE} at (58,13) size 0x2
     785                  RenderTableSection {THEAD} at (0,0) size 0x2
    786786                RenderInline {FONT} at (0,0) size 160x19
    787                   RenderText {#text} at (60,0) size 160x19
    788                     text run at (60,0) width 160: "All of this should be green."
     787                  RenderText {#text} at (58,0) size 160x19
     788                    text run at (58,0) width 160: "All of this should be green."
    789789                RenderText {#text} at (0,0) size 0x0
    790790      RenderBlock (anonymous) at (0,3375) size 769x3 [color=#008000]
    791791        RenderBlock {HR} at (0,0) size 769x2 [border: (1px inset #008000)]
    792792      RenderBlock (anonymous) at (0,3385) size 769x21
    793         RenderInline {FONT} at (0,0) size 188x19 [color=#008000]
    794           RenderInline {FONT} at (0,0) size 188x19
    795             RenderInline {FONT} at (0,0) size 188x19
    796               RenderInline {FONT} at (0,0) size 188x19
     793        RenderInline {FONT} at (0,0) size 186x19 [color=#008000]
     794          RenderInline {FONT} at (0,0) size 186x19
     795            RenderInline {FONT} at (0,0) size 186x19
     796              RenderInline {FONT} at (0,0) size 186x19
    797797                RenderText {#text} at (0,0) size 26x19
    798798                  text run at (0,0) width 26: "TR: "
    799                 RenderInline {FONT} at (0,0) size 162x19
     799                RenderInline {FONT} at (0,0) size 160x19
    800800                  RenderInline {FONT} at (0,0) size 0x19
    801                   RenderTable {TABLE} at (26,11) size 2x4
    802                     RenderTableSection {TBODY} at (0,0) size 2x4
    803                       RenderTableRow {TR} at (0,2) size 2x0
     801                  RenderTable {TABLE} at (26,11) size 0x4
     802                    RenderTableSection {TBODY} at (0,0) size 0x4
     803                      RenderTableRow {TR} at (0,2) size 0x0
    804804                  RenderInline {FONT} at (0,0) size 160x19
    805                     RenderText {#text} at (28,0) size 160x19
    806                       text run at (28,0) width 160: "All of this should be green."
     805                    RenderText {#text} at (26,0) size 160x19
     806                      text run at (26,0) width 160: "All of this should be green."
    807807                  RenderText {#text} at (0,0) size 0x0
    808808      RenderBlock (anonymous) at (0,3413) size 769x3 [color=#008000]
  • trunk/LayoutTests/platform/chromium-mac/fast/forms/file/file-input-disabled-expected.txt

    r117818 r128906  
    2020                  text run at (0,0) width 62: "Choose File"
    2121          RenderBR {BR} at (327,52) size 0x0
    22         RenderTable {TABLE} at (0,59) size 2x0
     22        RenderTable {TABLE} at (0,59) size 0x0
  • trunk/LayoutTests/platform/chromium-mac/tables/mozilla/bugs/bug113235-2-expected.txt

    r117818 r128906  
    1616            RenderText {#text} at (0,0) size 169x22
    1717              text run at (0,0) width 169: "Table of Contents"
    18         RenderTable {TABLE} at (0,39) size 2x5
    19           RenderTableSection {TBODY} at (0,0) size 2x4
    20             RenderTableRow {TR} at (0,2) size 2x0
     18        RenderTable {TABLE} at (0,39) size 0x5
     19          RenderTableSection {TBODY} at (0,0) size 0x4
     20            RenderTableRow {TR} at (0,2) size 0x0
    2121      RenderBlock (anonymous) at (0,74) size 784x1
    2222        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/chromium-win-xp/fast/forms/file-input-disabled-expected.txt

    r96516 r128906  
    2020                  text run at (0,0) width 69: "Choose File"
    2121          RenderBR {BR} at (324,59) size 0x0
    22         RenderTable {TABLE} at (0,67) size 2x0
     22        RenderTable {TABLE} at (0,67) size 0x0
  • trunk/LayoutTests/platform/chromium-win/fast/forms/file/file-input-disabled-expected.txt

    r102411 r128906  
    2020                  text run at (0,0) width 69: "Choose File"
    2121          RenderBR {BR} at (324,59) size 0x0
    22         RenderTable {TABLE} at (0,67) size 2x0
     22        RenderTable {TABLE} at (0,67) size 0x0
  • trunk/LayoutTests/platform/chromium-win/fast/invalid/017-expected.txt

    r96509 r128906  
    1010          RenderText {#text} at (0,0) size 147x19
    1111            text run at (0,0) width 147: "You should see this text."
    12       RenderTable {TABLE} at (0,36) size 2x4
    13         RenderTableSection {TBODY} at (0,0) size 2x4
    14           RenderTableRow {TR} at (0,2) size 2x0
     12      RenderTable {TABLE} at (0,36) size 0x4
     13        RenderTableSection {TBODY} at (0,0) size 0x4
     14          RenderTableRow {TR} at (0,2) size 0x0
  • trunk/LayoutTests/platform/chromium-win/fast/invalid/018-expected.txt

    r96509 r128906  
    1414            RenderTableCell {TD} at (2,2) size 2x2 [r=0 c=0 rs=1 cs=1]
    1515          RenderTableRow {TR} at (0,6) size 6x0
    16       RenderTable {TABLE} at (0,8) size 2x4
    17         RenderTableSection {TBODY} at (0,0) size 2x4
    18           RenderTableRow {TR} at (0,2) size 2x0
     16      RenderTable {TABLE} at (0,8) size 0x4
     17        RenderTableSection {TBODY} at (0,0) size 0x4
     18          RenderTableRow {TR} at (0,2) size 0x0
    1919      RenderBlock (anonymous) at (0,12) size 784x0
    2020        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/chromium-win/fast/invalid/020-expected.txt

    r96509 r128906  
    77        RenderText {#text} at (0,0) size 27x19
    88          text run at (0,0) width 27: "Text"
    9       RenderTable {table} at (0,20) size 2x0
     9      RenderTable {table} at (0,20) size 0x0
    1010      RenderTable at (0,20) size 0x0
    1111        RenderTableCol {colgroup} at (0,0) size 0x0
  • trunk/LayoutTests/platform/chromium-win/fast/invalid/table-inside-stray-table-content-expected.txt

    r96509 r128906  
    1010        RenderBlock {DIV} at (2,2) size 780x50 [border: (10px solid #008000)]
    1111          RenderBlock {DIV} at (10,10) size 760x14 [border: (2px solid #0000FF)]
    12         RenderTable {TABLE} at (2,52) size 2x0
     12        RenderTable {TABLE} at (2,52) size 0x0
    1313        RenderTable {TABLE} at (2,52) size 46x32 [border: (2px outset #808080)]
    1414          RenderTableSection {TBODY} at (2,2) size 42x28
  • trunk/LayoutTests/platform/chromium-win/tables/mozilla/bugs/bug113235-2-expected.txt

    r117821 r128906  
    1616            RenderText {#text} at (0,0) size 169x24
    1717              text run at (0,0) width 169: "Table of Contents"
    18         RenderTable {TABLE} at (0,41) size 2x5
    19           RenderTableSection {TBODY} at (0,0) size 2x4
    20             RenderTableRow {TR} at (0,2) size 2x0
     18        RenderTable {TABLE} at (0,41) size 0x5
     19          RenderTableSection {TBODY} at (0,0) size 0x4
     20            RenderTableRow {TR} at (0,2) size 0x0
    2121      RenderBlock (anonymous) at (0,77) size 784x1
    2222        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/chromium-win/tables/mozilla/bugs/bug23994-expected.txt

    r96509 r128906  
    77        RenderText {#text} at (0,0) size 9x19
    88          text run at (0,0) width 9: ">"
    9       RenderTable {TABLE} at (0,20) size 2x0
    10         RenderBlock {CAPTION} at (0,0) size 2x0
     9      RenderTable {TABLE} at (0,20) size 0x0
     10        RenderBlock {CAPTION} at (0,0) size 0x0
  • trunk/LayoutTests/platform/chromium-win/tables/mozilla/bugs/bug56405-expected.txt

    r96509 r128906  
    44  RenderBlock {HTML} at (0,0) size 785x1140
    55    RenderBody {BODY} at (8,8) size 769x1124
    6       RenderTable {TABLE} at (0,0) size 2x1078
     6      RenderTable {TABLE} at (0,0) size 0x1078
    77      RenderTable {TABLE} at (0,1078) size 98x26
    88        RenderTableSection {TBODY} at (0,0) size 98x26
  • trunk/LayoutTests/platform/chromium-win/tables/mozilla/bugs/bug56563-expected.txt

    r96509 r128906  
    1414        RenderImage {IMG} at (0,0) size 126x40
    1515        RenderText {#text} at (0,0) size 0x0
    16       RenderTable {TABLE} at (0,92) size 4x2 [border: (1px outset #808080)]
     16      RenderTable {TABLE} at (0,92) size 2x2 [border: (1px outset #808080)]
  • trunk/LayoutTests/platform/efl/fast/forms/file/file-input-disabled-expected.txt

    r126750 r128906  
    2020                  text run at (0,0) width 87: "Choose File"
    2121          RenderBR {BR} at (396,66) size 0x0
    22         RenderTable {TABLE} at (0,83) size 2x0
     22        RenderTable {TABLE} at (0,83) size 0x0
  • trunk/LayoutTests/platform/efl/fast/invalid/017-expected.txt

    r106181 r128906  
    1010          RenderText {#text} at (0,0) size 155x19
    1111            text run at (0,0) width 155: "You should see this text."
    12       RenderTable {TABLE} at (0,34) size 2x4
    13         RenderTableSection {TBODY} at (0,0) size 2x4
    14           RenderTableRow {TR} at (0,2) size 2x0
     12      RenderTable {TABLE} at (0,34) size 0x4
     13        RenderTableSection {TBODY} at (0,0) size 0x4
     14          RenderTableRow {TR} at (0,2) size 0x0
  • trunk/LayoutTests/platform/efl/fast/invalid/018-expected.txt

    r106181 r128906  
    1414            RenderTableCell {TD} at (2,2) size 2x2 [r=0 c=0 rs=1 cs=1]
    1515          RenderTableRow {TR} at (0,6) size 6x0
    16       RenderTable {TABLE} at (0,8) size 2x4
    17         RenderTableSection {TBODY} at (0,0) size 2x4
    18           RenderTableRow {TR} at (0,2) size 2x0
     16      RenderTable {TABLE} at (0,8) size 0x4
     17        RenderTableSection {TBODY} at (0,0) size 0x4
     18          RenderTableRow {TR} at (0,2) size 0x0
    1919      RenderBlock (anonymous) at (0,12) size 784x0
    2020        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/efl/fast/invalid/020-expected.txt

    r106181 r128906  
    77        RenderText {#text} at (0,0) size 29x19
    88          text run at (0,0) width 29: "Text"
    9       RenderTable {table} at (0,18) size 2x0
     9      RenderTable {table} at (0,18) size 0x0
    1010      RenderTable at (0,18) size 0x0
    1111        RenderTableCol {colgroup} at (0,0) size 0x0
  • trunk/LayoutTests/platform/efl/fast/invalid/table-inside-stray-table-content-expected.txt

    r106181 r128906  
    1010        RenderBlock {DIV} at (2,2) size 780x50 [border: (10px solid #008000)]
    1111          RenderBlock {DIV} at (10,10) size 760x14 [border: (2px solid #0000FF)]
    12         RenderTable {TABLE} at (2,52) size 2x0
     12        RenderTable {TABLE} at (2,52) size 0x0
    1313        RenderTable {TABLE} at (2,52) size 48x30 [border: (2px outset #808080)]
    1414          RenderTableSection {TBODY} at (2,2) size 44x26
  • trunk/LayoutTests/platform/efl/fast/invalid/table-residual-style-crash-expected.txt

    r97999 r128906  
    1111      RenderBlock (anonymous) at (0,0) size 784x0
    1212        RenderInline {FONT} at (0,0) size 0x0
    13       RenderTable {TABLE} at (0,0) size 2x0
     13      RenderTable {TABLE} at (0,0) size 0x0
  • trunk/LayoutTests/platform/efl/tables/mozilla/bugs/bug23994-expected.txt

    r106182 r128906  
    77        RenderText {#text} at (0,0) size 9x19
    88          text run at (0,0) width 9: ">"
    9       RenderTable {TABLE} at (0,18) size 2x0
    10         RenderBlock {CAPTION} at (0,0) size 2x0
     9      RenderTable {TABLE} at (0,18) size 0x0
     10        RenderBlock {CAPTION} at (0,0) size 0x0
  • trunk/LayoutTests/platform/efl/tables/mozilla/bugs/bug56405-expected.txt

    r106182 r128906  
    44  RenderBlock {HTML} at (0,0) size 785x1136
    55    RenderBody {BODY} at (8,8) size 769x1120
    6       RenderTable {TABLE} at (0,0) size 2x1078
     6      RenderTable {TABLE} at (0,0) size 0x1078
    77      RenderTable {TABLE} at (0,1078) size 102x24
    88        RenderTableSection {TBODY} at (0,0) size 102x24
  • trunk/LayoutTests/platform/gtk/fast/forms/file/file-input-disabled-expected.txt

    r114833 r128906  
    2020                  text run at (0,0) width 87: "Choose File"
    2121          RenderBR {BR} at (396,61) size 0x0
    22         RenderTable {TABLE} at (0,71) size 2x0
     22        RenderTable {TABLE} at (0,71) size 0x0
  • trunk/LayoutTests/platform/gtk/fast/invalid/017-expected.txt

    r114836 r128906  
    1010          RenderText {#text} at (0,0) size 155x19
    1111            text run at (0,0) width 155: "You should see this text."
    12       RenderTable {TABLE} at (0,34) size 2x4
    13         RenderTableSection {TBODY} at (0,0) size 2x4
    14           RenderTableRow {TR} at (0,2) size 2x0
     12      RenderTable {TABLE} at (0,34) size 0x4
     13        RenderTableSection {TBODY} at (0,0) size 0x4
     14          RenderTableRow {TR} at (0,2) size 0x0
  • trunk/LayoutTests/platform/gtk/fast/invalid/018-expected.txt

    r114836 r128906  
    1414            RenderTableCell {TD} at (2,2) size 2x2 [r=0 c=0 rs=1 cs=1]
    1515          RenderTableRow {TR} at (0,6) size 6x0
    16       RenderTable {TABLE} at (0,8) size 2x4
    17         RenderTableSection {TBODY} at (0,0) size 2x4
    18           RenderTableRow {TR} at (0,2) size 2x0
     16      RenderTable {TABLE} at (0,8) size 0x4
     17        RenderTableSection {TBODY} at (0,0) size 0x4
     18          RenderTableRow {TR} at (0,2) size 0x0
    1919      RenderBlock (anonymous) at (0,12) size 784x0
    2020        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/gtk/fast/invalid/020-expected.txt

    r114836 r128906  
    77        RenderText {#text} at (0,0) size 29x19
    88          text run at (0,0) width 29: "Text"
    9       RenderTable {table} at (0,18) size 2x0
     9      RenderTable {table} at (0,18) size 0x0
    1010      RenderTable at (0,18) size 0x0
    1111        RenderTableCol {colgroup} at (0,0) size 0x0
  • trunk/LayoutTests/platform/gtk/fast/invalid/table-inside-stray-table-content-expected.txt

    r114836 r128906  
    1010        RenderBlock {DIV} at (2,2) size 780x50 [border: (10px solid #008000)]
    1111          RenderBlock {DIV} at (10,10) size 760x14 [border: (2px solid #0000FF)]
    12         RenderTable {TABLE} at (2,52) size 2x0
     12        RenderTable {TABLE} at (2,52) size 0x0
    1313        RenderTable {TABLE} at (2,52) size 48x30 [border: (2px outset #808080)]
    1414          RenderTableSection {TBODY} at (2,2) size 44x26
  • trunk/LayoutTests/platform/gtk/fast/invalid/table-residual-style-crash-expected.txt

    r96509 r128906  
    1111      RenderBlock (anonymous) at (0,0) size 784x0
    1212        RenderInline {FONT} at (0,0) size 0x0
    13       RenderTable {TABLE} at (0,0) size 2x0
     13      RenderTable {TABLE} at (0,0) size 0x0
  • trunk/LayoutTests/platform/gtk/tables/mozilla/bugs/bug23994-expected.txt

    r114836 r128906  
    77        RenderText {#text} at (0,0) size 9x19
    88          text run at (0,0) width 9: ">"
    9       RenderTable {TABLE} at (0,18) size 2x0
    10         RenderBlock {CAPTION} at (0,0) size 2x0
     9      RenderTable {TABLE} at (0,18) size 0x0
     10        RenderBlock {CAPTION} at (0,0) size 0x0
  • trunk/LayoutTests/platform/gtk/tables/mozilla/bugs/bug56405-expected.txt

    r114836 r128906  
    44  RenderBlock {HTML} at (0,0) size 785x1136
    55    RenderBody {BODY} at (8,8) size 769x1120
    6       RenderTable {TABLE} at (0,0) size 2x1078
     6      RenderTable {TABLE} at (0,0) size 0x1078
    77      RenderTable {TABLE} at (0,1078) size 102x24
    88        RenderTableSection {TBODY} at (0,0) size 102x24
  • trunk/LayoutTests/platform/mac/fast/forms/file/file-input-disabled-expected.txt

    r96918 r128906  
    2020                  text run at (0,0) width 62: "Choose File"
    2121          RenderBR {BR} at (327,52) size 0x0
    22         RenderTable {TABLE} at (0,59) size 2x0
     22        RenderTable {TABLE} at (0,59) size 0x0
  • trunk/LayoutTests/platform/mac/fast/invalid/017-expected.txt

    r96573 r128906  
    1010          RenderText {#text} at (0,0) size 155x18
    1111            text run at (0,0) width 155: "You should see this text."
    12       RenderTable {TABLE} at (0,34) size 2x4
    13         RenderTableSection {TBODY} at (0,0) size 2x4
    14           RenderTableRow {TR} at (0,2) size 2x0
     12      RenderTable {TABLE} at (0,34) size 0x4
     13        RenderTableSection {TBODY} at (0,0) size 0x4
     14          RenderTableRow {TR} at (0,2) size 0x0
  • trunk/LayoutTests/platform/mac/fast/invalid/018-expected.txt

    r96573 r128906  
    1414            RenderTableCell {TD} at (2,2) size 2x2 [r=0 c=0 rs=1 cs=1]
    1515          RenderTableRow {TR} at (0,6) size 6x0
    16       RenderTable {TABLE} at (0,8) size 2x4
    17         RenderTableSection {TBODY} at (0,0) size 2x4
    18           RenderTableRow {TR} at (0,2) size 2x0
     16      RenderTable {TABLE} at (0,8) size 0x4
     17        RenderTableSection {TBODY} at (0,0) size 0x4
     18          RenderTableRow {TR} at (0,2) size 0x0
    1919      RenderBlock (anonymous) at (0,12) size 784x0
    2020        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/mac/fast/invalid/020-expected.txt

    r96573 r128906  
    77        RenderText {#text} at (0,0) size 29x18
    88          text run at (0,0) width 29: "Text"
    9       RenderTable {table} at (0,18) size 2x0
     9      RenderTable {table} at (0,18) size 0x0
    1010      RenderTable at (0,18) size 0x0
    1111        RenderTableCol {colgroup} at (0,0) size 0x0
  • trunk/LayoutTests/platform/mac/fast/invalid/table-inside-stray-table-content-expected.txt

    r96573 r128906  
    1010        RenderBlock {DIV} at (2,2) size 780x50 [border: (10px solid #008000)]
    1111          RenderBlock {DIV} at (10,10) size 760x14 [border: (2px solid #0000FF)]
    12         RenderTable {TABLE} at (2,52) size 2x0
     12        RenderTable {TABLE} at (2,52) size 0x0
    1313        RenderTable {TABLE} at (2,52) size 48x30 [border: (2px outset #808080)]
    1414          RenderTableSection {TBODY} at (2,2) size 44x26
  • trunk/LayoutTests/platform/mac/fast/invalid/table-residual-style-crash-expected.txt

    r96509 r128906  
    1111      RenderBlock (anonymous) at (0,0) size 784x0
    1212        RenderInline {FONT} at (0,0) size 0x0
    13       RenderTable {TABLE} at (0,0) size 2x0
     13      RenderTable {TABLE} at (0,0) size 0x0
  • trunk/LayoutTests/platform/mac/tables/mozilla/bugs/bug113235-2-expected.txt

    r96573 r128906  
    1616            RenderText {#text} at (0,0) size 169x22
    1717              text run at (0,0) width 169: "Table of Contents"
    18         RenderTable {TABLE} at (0,39) size 2x4
    19           RenderTableSection {TBODY} at (0,0) size 2x4
    20             RenderTableRow {TR} at (0,2) size 2x0
     18        RenderTable {TABLE} at (0,39) size 0x4
     19          RenderTableSection {TBODY} at (0,0) size 0x4
     20            RenderTableRow {TR} at (0,2) size 0x0
    2121      RenderBlock (anonymous) at (0,74) size 784x0
    2222        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/mac/tables/mozilla/bugs/bug23994-expected.txt

    r96573 r128906  
    77        RenderText {#text} at (0,0) size 9x18
    88          text run at (0,0) width 9: ">"
    9       RenderTable {TABLE} at (0,18) size 2x0
    10         RenderBlock {CAPTION} at (0,0) size 2x0
     9      RenderTable {TABLE} at (0,18) size 0x0
     10        RenderBlock {CAPTION} at (0,0) size 0x0
  • trunk/LayoutTests/platform/mac/tables/mozilla/bugs/bug56405-expected.txt

    r96573 r128906  
    44  RenderBlock {HTML} at (0,0) size 785x1136
    55    RenderBody {BODY} at (8,8) size 769x1120
    6       RenderTable {TABLE} at (0,0) size 2x1078
     6      RenderTable {TABLE} at (0,0) size 0x1078
    77      RenderTable {TABLE} at (0,1078) size 102x24
    88        RenderTableSection {TBODY} at (0,0) size 102x24
  • trunk/LayoutTests/platform/mac/tables/mozilla/bugs/bug56563-expected.txt

    r96573 r128906  
    1414        RenderImage {IMG} at (0,0) size 126x40
    1515        RenderText {#text} at (0,0) size 0x0
    16       RenderTable {TABLE} at (0,90) size 4x2 [border: (1px outset #808080)]
     16      RenderTable {TABLE} at (0,90) size 2x2 [border: (1px outset #808080)]
  • trunk/LayoutTests/platform/qt/fast/forms/file/file-input-disabled-expected.txt

    r122879 r128906  
    2020                  text run at (0,0) width 73: "Choose File"
    2121          RenderBR {BR} at (358,62) size 0x0
    22         RenderTable {TABLE} at (0,74) size 2x0
     22        RenderTable {TABLE} at (0,74) size 0x0
  • trunk/LayoutTests/platform/qt/fast/invalid/017-expected.txt

    r123100 r128906  
    1010          RenderText {#text} at (0,0) size 147x19
    1111            text run at (0,0) width 147: "You should see this text."
    12       RenderTable {TABLE} at (0,35) size 2x4
    13         RenderTableSection {TBODY} at (0,0) size 2x4
    14           RenderTableRow {TR} at (0,2) size 2x0
     12      RenderTable {TABLE} at (0,35) size 0x4
     13        RenderTableSection {TBODY} at (0,0) size 0x4
     14          RenderTableRow {TR} at (0,2) size 0x0
  • trunk/LayoutTests/platform/qt/fast/invalid/018-expected.txt

    r123100 r128906  
    1414            RenderTableCell {TD} at (2,2) size 2x2 [r=0 c=0 rs=1 cs=1]
    1515          RenderTableRow {TR} at (0,6) size 6x0
    16       RenderTable {TABLE} at (0,8) size 2x4
    17         RenderTableSection {TBODY} at (0,0) size 2x4
    18           RenderTableRow {TR} at (0,2) size 2x0
     16      RenderTable {TABLE} at (0,8) size 0x4
     17        RenderTableSection {TBODY} at (0,0) size 0x4
     18          RenderTableRow {TR} at (0,2) size 0x0
    1919      RenderBlock (anonymous) at (0,12) size 784x0
    2020        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/qt/fast/invalid/020-expected.txt

    r124189 r128906  
    77        RenderText {#text} at (0,0) size 27x19
    88          text run at (0,0) width 27: "Text"
    9       RenderTable {table} at (0,19) size 2x0
     9      RenderTable {table} at (0,19) size 0x0
    1010      RenderTable at (0,19) size 0x0
    1111        RenderTableCol {colgroup} at (0,0) size 0x0
  • trunk/LayoutTests/platform/qt/fast/invalid/table-inside-stray-table-content-expected.txt

    r123100 r128906  
    1010        RenderBlock {DIV} at (2,2) size 780x50 [border: (10px solid #008000)]
    1111          RenderBlock {DIV} at (10,10) size 760x14 [border: (2px solid #0000FF)]
    12         RenderTable {TABLE} at (2,52) size 2x0
     12        RenderTable {TABLE} at (2,52) size 0x0
    1313        RenderTable {TABLE} at (2,52) size 46x31 [border: (2px outset #808080)]
    1414          RenderTableSection {TBODY} at (2,2) size 42x27
  • trunk/LayoutTests/platform/qt/fast/invalid/table-residual-style-crash-expected.txt

    r96509 r128906  
    1111      RenderBlock (anonymous) at (0,0) size 784x0
    1212        RenderInline {FONT} at (0,0) size 0x0
    13       RenderTable {TABLE} at (0,0) size 2x0
     13      RenderTable {TABLE} at (0,0) size 0x0
  • trunk/LayoutTests/platform/qt/tables/mozilla/bugs/bug113235-2-expected.txt

    r122719 r128906  
    1616            RenderText {#text} at (0,0) size 169x24
    1717              text run at (0,0) width 169: "Table of Contents"
    18         RenderTable {TABLE} at (0,40) size 2x4
    19           RenderTableSection {TBODY} at (0,0) size 2x4
    20             RenderTableRow {TR} at (0,2) size 2x0
     18        RenderTable {TABLE} at (0,40) size 0x4
     19          RenderTableSection {TBODY} at (0,0) size 0x4
     20            RenderTableRow {TR} at (0,2) size 0x0
    2121      RenderBlock (anonymous) at (0,75) size 784x0
    2222        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/platform/qt/tables/mozilla/bugs/bug23994-expected.txt

    r123670 r128906  
    77        RenderText {#text} at (0,0) size 9x19
    88          text run at (0,0) width 9: ">"
    9       RenderTable {TABLE} at (0,19) size 2x0
    10         RenderBlock {CAPTION} at (0,0) size 2x0
     9      RenderTable {TABLE} at (0,19) size 0x0
     10        RenderBlock {CAPTION} at (0,0) size 0x0
  • trunk/LayoutTests/platform/qt/tables/mozilla/bugs/bug56405-expected.txt

    r123670 r128906  
    44  RenderBlock {HTML} at (0,0) size 784x1138
    55    RenderBody {BODY} at (8,8) size 768x1122
    6       RenderTable {TABLE} at (0,0) size 2x1078
     6      RenderTable {TABLE} at (0,0) size 0x1078
    77      RenderTable {TABLE} at (0,1078) size 98x25
    88        RenderTableSection {TBODY} at (0,0) size 98x25
  • trunk/LayoutTests/platform/qt/tables/mozilla/bugs/bug56563-expected.txt

    r122719 r128906  
    1414        RenderImage {IMG} at (0,0) size 126x40
    1515        RenderText {#text} at (0,0) size 0x0
    16       RenderTable {TABLE} at (0,91) size 4x2 [border: (1px outset #808080)]
     16      RenderTable {TABLE} at (0,91) size 2x2 [border: (1px outset #808080)]
  • trunk/LayoutTests/platform/win/fast/forms/file-input-disabled-expected.txt

    r80778 r128906  
    2020                  text run at (0,0) width 62: "Choose File"
    2121          RenderBR {BR} at (327,52) size 0x0
    22         RenderTable {TABLE} at (0,59) size 4x0
     22        RenderTable {TABLE} at (0,59) size 0x0
  • trunk/LayoutTests/platform/win/fast/forms/file/file-input-disabled-expected.txt

    r119166 r128906  
    2020                  text run at (0,0) width 62: "Choose File"
    2121          RenderBR {BR} at (327,52) size 0x0
    22         RenderTable {TABLE} at (0,59) size 2x0
     22        RenderTable {TABLE} at (0,59) size 0x0
  • trunk/LayoutTests/tables/mozilla/bugs/bug113235-2-expected.txt

    r117900 r128906  
    1616            RenderText {#text} at (0,0) size 169x24
    1717              text run at (0,0) width 169: "Table of Contents"
    18         RenderTable {TABLE} at (0,39) size 2x4
    19           RenderTableSection {TBODY} at (0,0) size 2x4
    20             RenderTableRow {TR} at (0,2) size 2x0
     18        RenderTable {TABLE} at (0,39) size 0x4
     19          RenderTableSection {TBODY} at (0,0) size 0x4
     20            RenderTableRow {TR} at (0,2) size 0x0
    2121      RenderBlock (anonymous) at (0,74) size 784x0
    2222        RenderInline {FONT} at (0,0) size 0x0
  • trunk/LayoutTests/tables/mozilla/bugs/bug56563-expected.txt

    r116839 r128906  
    1414        RenderImage {IMG} at (0,0) size 126x40
    1515        RenderText {#text} at (0,0) size 0x0
    16       RenderTable {TABLE} at (0,90) size 4x2 [border: (1px outset #808080)]
     16      RenderTable {TABLE} at (0,90) size 2x2 [border: (1px outset #808080)]
  • trunk/Source/WebCore/ChangeLog

    r128905 r128906  
     12012-09-18  Julien Chaffraix  <jchaffraix@webkit.org>
     2
     3        Tables without any descendant and auto logical width should have a 0px logical width
     4        https://bugs.webkit.org/show_bug.cgi?id=95521
     5
     6        Reviewed by Abhishek Arya.
     7
     8        The code would wrongly add the border-spacing in the row direction to the table's logical
     9        width even if we didn't have a column. The new behavior matches FireFox and Opera. IE
     10        matches our old behavior for inline tables but our new behavior for normal tables which
     11        is a bug on their side.
     12
     13        Tests: fast/table/empty-table-should-take-no-space.html
     14               fast/table/fixed-table-layout/empty-table-should-take-no-space-fixed-layout.html
     15
     16        * rendering/RenderTable.h:
     17        (WebCore::RenderTable::borderSpacingInRowDirection):
     18        Added this new helper function to return the right border-spacing. Added a FIXME as the code always
     19        return the horizontal dimension which is wrong in vertical-writing mode.
     20
     21        (WebCore::RenderTable::bordersPaddingAndSpacingInRowDirection):
     22        Changed to call borderSpacingInRowDirection. Added a comment as to why we don't add border-spacing on
     23        border-collapse: separate tables.
     24
    1252012-09-18  Zan Dobersek  <zandobersek@gmail.com>
    226
  • trunk/Source/WebCore/rendering/RenderTable.h

    r128110 r128906  
    4848    int getColumnPos(unsigned col) const { return m_columnPos[col]; }
    4949
     50    // Per CSS 3 writing-mode: "The first and second values of the 'border-spacing' property represent spacing between columns
     51    // and rows respectively, not necessarily the horizontal and vertical spacing respectively".
    5052    int hBorderSpacing() const { return m_hSpacing; }
    5153    int vBorderSpacing() const { return m_vSpacing; }
     
    179181    }
    180182
     183    LayoutUnit borderSpacingInRowDirection() const
     184    {
     185        if (unsigned effectiveColumnCount = numEffCols())
     186            return static_cast<LayoutUnit>(effectiveColumnCount + 1) * hBorderSpacing();
     187
     188        return ZERO_LAYOUT_UNIT;
     189    }
     190
    181191    LayoutUnit bordersPaddingAndSpacingInRowDirection() const
    182192    {
     193        // 'border-spacing' only applies to separate borders (see 17.6.1 The separated borders model).
    183194        return borderStart() + borderEnd() +
    184                (collapseBorders() ? ZERO_LAYOUT_UNIT : (paddingStart() + paddingEnd() + static_cast<LayoutUnit>(numEffCols() + 1) * hBorderSpacing()));
     195               (collapseBorders() ? ZERO_LAYOUT_UNIT : (paddingStart() + paddingEnd() + borderSpacingInRowDirection()));
    185196    }
    186197
Note: See TracChangeset for help on using the changeset viewer.