Floor cell widths in AutoTableLayout::recalcColumn
authoreae@chromium.org <eae@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Mar 2013 13:40:57 +0000 (13:40 +0000)
committereae@chromium.org <eae@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 22 Mar 2013 13:40:57 +0000 (13:40 +0000)
commit6174b552e17a301981792b77a23fcc420c1caac8
treeb504014fea06e49045e4f238d0051be897aa017a
parent62746b11a2a8370cfb4164ee1b796efe20c878cd
Floor cell widths in AutoTableLayout::recalcColumn
https://bugs.webkit.org/show_bug.cgi?id=112922

Reviewed by Levi Weintraub.

Source/WebCore:

Table cells widths are floored to ensure even distribution of available
space across columns. The code path that computes the width of the
_table_ failed to take this into account and thus can over report the
total width of the table.

Test: fast/sub-pixel/table-with-subpixel-cell-size.html

* rendering/AutoTableLayout.cpp:
(WebCore::AutoTableLayout::recalcColumn): Floor width to match column sizing logic.

LayoutTests:

Add test for table with subpixel cells and update existing baselines as needed.

* fast/sub-pixel/table-with-subpixel-cell-size-expected.html: Added.
* fast/sub-pixel/table-with-subpixel-cell-size.html: Added.
* platform/chromium-linux/fast/text/international/bidi-layout-across-linebreak-expected.png:
* platform/chromium-linux/tables/mozilla_expected_failures/bugs/bug89315-expected.png:
* platform/chromium-win/fast/text/international/bidi-layout-across-linebreak-expected.txt:
* platform/chromium-win/fast/transforms/bounding-rect-zoom-expected.txt:
* platform/chromium-win/tables/mozilla_expected_failures/bugs/bug89315-expected.txt:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@146600 268f45cc-cd09-0410-ab3c-d52691b4dbfc
LayoutTests/ChangeLog
LayoutTests/fast/sub-pixel/table-with-subpixel-cell-size-expected.html [new file with mode: 0644]
LayoutTests/fast/sub-pixel/table-with-subpixel-cell-size.html [new file with mode: 0644]
LayoutTests/platform/chromium-linux/fast/text/international/bidi-layout-across-linebreak-expected.png
LayoutTests/platform/chromium-linux/tables/mozilla_expected_failures/bugs/bug89315-expected.png
LayoutTests/platform/chromium-win/fast/text/international/bidi-layout-across-linebreak-expected.txt
LayoutTests/platform/chromium-win/fast/transforms/bounding-rect-zoom-expected.txt
LayoutTests/platform/chromium-win/tables/mozilla_expected_failures/bugs/bug89315-expected.txt
Source/WebCore/ChangeLog
Source/WebCore/rendering/AutoTableLayout.cpp