Bug 112922 - Floor cell widths in AutoTableLayout::recalcColumn
Summary: Floor cell widths in AutoTableLayout::recalcColumn
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Emil A Eklund
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-21 08:59 PDT by Emil A Eklund
Modified: 2013-03-22 06:43 PDT (History)
8 users (show)

See Also:


Attachments
Patch (419.00 KB, patch)
2013-03-21 09:56 PDT, Emil A Eklund
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-14 (653.68 KB, application/zip)
2013-03-21 11:25 PDT, Build Bot
no flags Details
Archive of layout-test-results from gce-cr-linux-04 (618.99 KB, application/zip)
2013-03-21 11:52 PDT, WebKit Review Bot
no flags Details
Archive of layout-test-results from webkit-ews-07 (861.67 KB, application/zip)
2013-03-21 16:48 PDT, Build Bot
no flags Details
Archive of layout-test-results from webkit-ews-02 (742.63 KB, application/zip)
2013-03-21 17:52 PDT, Build Bot
no flags Details
Patch for landing (459.82 KB, patch)
2013-03-22 06:04 PDT, Emil A Eklund
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Emil A Eklund 2013-03-21 08:59:35 PDT
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.

Donstream chromium bug: https://code.google.com/p/chromium/issues/detail?id=179365
Comment 1 Emil A Eklund 2013-03-21 09:56:09 PDT
Created attachment 194284 [details]
Patch
Comment 2 Levi Weintraub 2013-03-21 10:59:35 PDT
Comment on attachment 194284 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=194284&action=review

> Source/WebCore/ChangeLog:11
> +        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.

Whoops!! Thanks Emil :)

> LayoutTests/ChangeLog:11
> +        * fast/sub-pixel/table-with-subpixel-cell-size-expected.html: Added.
> +        * fast/sub-pixel/table-with-subpixel-cell-size.html: Added.

Why no entries for the the changed expectations? Also, I presume this will trigger failures on other platforms as well?

> LayoutTests/platform/chromium-win/tables/mozilla_expected_failures/bugs/bug89315-expected.txt:10
>          RenderTable {TABLE} at (67,20) size 474x4525

Man this output sucks :(
Comment 3 Build Bot 2013-03-21 11:25:11 PDT
Comment on attachment 194284 [details]
Patch

Attachment 194284 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-commit-queue.appspot.com/results/17147616

New failing tests:
fast/transforms/bounding-rect-zoom.html
tables/mozilla_expected_failures/bugs/bug89315.html
fast/text/international/bidi-layout-across-linebreak.html
Comment 4 Build Bot 2013-03-21 11:25:13 PDT
Created attachment 194295 [details]
Archive of layout-test-results from webkit-ews-14

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: webkit-ews-14  Port: <class 'webkitpy.common.config.ports.MacWK2Port'>  Platform: Mac OS X 10.8.2
Comment 5 WebKit Review Bot 2013-03-21 11:52:09 PDT
Comment on attachment 194284 [details]
Patch

Attachment 194284 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://webkit-commit-queue.appspot.com/results/17125828

New failing tests:
fast/text/international/bidi-layout-across-linebreak.html
Comment 6 WebKit Review Bot 2013-03-21 11:52:13 PDT
Created attachment 194308 [details]
Archive of layout-test-results from gce-cr-linux-04

The attached test failures were seen while running run-webkit-tests on the chromium-ews.
Bot: gce-cr-linux-04  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-3.3.8-gcg-201212281604-x86_64-with-GCEL-10.04-gcel_10.04
Comment 7 Build Bot 2013-03-21 16:48:44 PDT
Comment on attachment 194284 [details]
Patch

Attachment 194284 [details] did not pass mac-ews (mac):
Output: http://webkit-commit-queue.appspot.com/results/17212874

New failing tests:
fast/transforms/bounding-rect-zoom.html
tables/mozilla_expected_failures/bugs/bug89315.html
svg/custom/marker-orient-auto.html
fast/text/international/bidi-layout-across-linebreak.html
Comment 8 Build Bot 2013-03-21 16:48:46 PDT
Created attachment 194385 [details]
Archive of layout-test-results from webkit-ews-07

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-07  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.2
Comment 9 Build Bot 2013-03-21 17:52:34 PDT
Comment on attachment 194284 [details]
Patch

Attachment 194284 [details] did not pass mac-ews (mac):
Output: http://webkit-commit-queue.appspot.com/results/17212891

New failing tests:
fast/transforms/bounding-rect-zoom.html
tables/mozilla_expected_failures/bugs/bug89315.html
svg/custom/marker-orient-auto.html
fast/text/international/bidi-layout-across-linebreak.html
Comment 10 Build Bot 2013-03-21 17:52:36 PDT
Created attachment 194403 [details]
Archive of layout-test-results from webkit-ews-02

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-02  Port: <class 'webkitpy.common.config.ports.MacPort'>  Platform: Mac OS X 10.8.2
Comment 11 Emil A Eklund 2013-03-22 06:04:45 PDT
Created attachment 194526 [details]
Patch for landing
Comment 12 WebKit Review Bot 2013-03-22 06:43:27 PDT
Comment on attachment 194526 [details]
Patch for landing

Clearing flags on attachment: 194526

Committed r146600: <http://trac.webkit.org/changeset/146600>
Comment 13 WebKit Review Bot 2013-03-22 06:43:32 PDT
All reviewed patches have been landed.  Closing bug.