LayoutTests:
authorantti <antti@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Apr 2007 11:32:29 +0000 (11:32 +0000)
committerantti <antti@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 27 Apr 2007 11:32:29 +0000 (11:32 +0000)
commit8d40a489b0d5973b3621631ebc20362815382129
treeaa0a3a2ed699d047f5a252d40215a979b75aa094
parent1644300922c3dc019f16965e94de8351b89f7cbd
LayoutTests:

        Reviewed by Darin, Hyatt.

        Test for http://bugs.webkit.org/show_bug.cgi?id=13081
        Empty table cells not handled correctly
        <rdar://problem/5067926>

        Added test case and checked in updated results for others. Changes represent
        progressions or are not visible in rendering output.

        * fast/block/positioning/negative-right-pos-expected.txt:
        * fast/css/acid2-expected.txt:
        * fast/css/acid2-pixel-expected.txt:
        * fast/css/percentage-non-integer-expected.checksum:
        * fast/css/percentage-non-integer-expected.png:
        * fast/css/percentage-non-integer-expected.txt:
        * fast/repaint/table-cell-move-expected.checksum:
        * fast/repaint/table-cell-move-expected.png:
        * fast/repaint/table-cell-move-expected.txt:
        * fast/table/012-expected.txt:
        * fast/table/032-expected.txt:
        * fast/table/empty-cells-expected.checksum: Added.
        * fast/table/empty-cells-expected.txt: Added.
        * fast/table/empty-cells.html: Added.
        * fast/table/empty-section-crash-expected.txt:
        * http/tests/misc/acid2-expected.txt:
        * http/tests/misc/acid2-pixel-expected.txt:
        * tables/mozilla/bugs/bug100334-expected.txt:
        * tables/mozilla/bugs/bug1188-expected.txt:
        * tables/mozilla/bugs/bug16012-expected.checksum:
        * tables/mozilla/bugs/bug16012-expected.png:
        * tables/mozilla/bugs/bug16012-expected.txt:
        * tables/mozilla/bugs/bug1818-6-expected.txt:
        * tables/mozilla/bugs/bug222336-expected.txt:
        * tables/mozilla/bugs/bug222467-expected.txt:
        * tables/mozilla/bugs/bug78162-expected.txt:
        * tables/mozilla/core/misc-expected.checksum:
        * tables/mozilla/core/misc-expected.png:
        * tables/mozilla/core/misc-expected.txt:
        * tables/mozilla_expected_failures/bugs/bug14007-1-expected.checksum:
        * tables/mozilla_expected_failures/bugs/bug14007-1-expected.png:
        * tables/mozilla_expected_failures/bugs/bug14007-1-expected.txt:
        * tables/mozilla_expected_failures/bugs/bug72393-expected.checksum:
        * tables/mozilla_expected_failures/bugs/bug72393-expected.png:
        * tables/mozilla_expected_failures/bugs/bug72393-expected.txt:
        * tables/mozilla_expected_failures/other/empty_cells-expected.txt:

WebCore:

        Reviewed by Darin, Hyatt.

        Fix http://bugs.webkit.org/show_bug.cgi?id=13081
        Empty table cells not handled correctly
        <rdar://problem/5067926>

        To match Firefox and IE, empty table cells with variable width need to be
        treated as special case. They generally get zero width.

        * rendering/AutoTableLayout.cpp:
        (WebCore::AutoTableLayout::recalcColumn):
        (WebCore::AutoTableLayout::calcEffectiveWidth):
        (WebCore::AutoTableLayout::layout):
        * rendering/AutoTableLayout.h:
        (WebCore::AutoTableLayout::Layout::Layout):
        * rendering/RenderStyle.h:
        (WebCore::RenderStyle::hasPadding):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@21145 268f45cc-cd09-0410-ab3c-d52691b4dbfc
41 files changed:
LayoutTests/ChangeLog
LayoutTests/fast/block/positioning/negative-right-pos-expected.txt
LayoutTests/fast/css/acid2-expected.txt
LayoutTests/fast/css/acid2-pixel-expected.txt
LayoutTests/fast/css/percentage-non-integer-expected.checksum
LayoutTests/fast/css/percentage-non-integer-expected.png
LayoutTests/fast/css/percentage-non-integer-expected.txt
LayoutTests/fast/repaint/table-cell-move-expected.checksum
LayoutTests/fast/repaint/table-cell-move-expected.png
LayoutTests/fast/repaint/table-cell-move-expected.txt
LayoutTests/fast/table/012-expected.txt
LayoutTests/fast/table/032-expected.txt
LayoutTests/fast/table/empty-cells-expected.checksum [new file with mode: 0644]
LayoutTests/fast/table/empty-cells-expected.txt [new file with mode: 0644]
LayoutTests/fast/table/empty-cells.html [new file with mode: 0644]
LayoutTests/fast/table/empty-section-crash-expected.txt
LayoutTests/http/tests/misc/acid2-expected.txt
LayoutTests/http/tests/misc/acid2-pixel-expected.txt
LayoutTests/tables/mozilla/bugs/bug100334-expected.txt
LayoutTests/tables/mozilla/bugs/bug1188-expected.txt
LayoutTests/tables/mozilla/bugs/bug16012-expected.checksum
LayoutTests/tables/mozilla/bugs/bug16012-expected.png
LayoutTests/tables/mozilla/bugs/bug16012-expected.txt
LayoutTests/tables/mozilla/bugs/bug1818-6-expected.txt
LayoutTests/tables/mozilla/bugs/bug222336-expected.txt
LayoutTests/tables/mozilla/bugs/bug222467-expected.txt
LayoutTests/tables/mozilla/bugs/bug78162-expected.txt
LayoutTests/tables/mozilla/core/misc-expected.checksum
LayoutTests/tables/mozilla/core/misc-expected.png
LayoutTests/tables/mozilla/core/misc-expected.txt
LayoutTests/tables/mozilla_expected_failures/bugs/bug14007-1-expected.checksum
LayoutTests/tables/mozilla_expected_failures/bugs/bug14007-1-expected.png
LayoutTests/tables/mozilla_expected_failures/bugs/bug14007-1-expected.txt
LayoutTests/tables/mozilla_expected_failures/bugs/bug72393-expected.checksum
LayoutTests/tables/mozilla_expected_failures/bugs/bug72393-expected.png
LayoutTests/tables/mozilla_expected_failures/bugs/bug72393-expected.txt
LayoutTests/tables/mozilla_expected_failures/other/empty_cells-expected.txt
WebCore/ChangeLog
WebCore/rendering/AutoTableLayout.cpp
WebCore/rendering/AutoTableLayout.h
WebCore/rendering/RenderStyle.h