Bug 87536

Summary: REGRESSION(r111742): box-sizing: border-box doesn't work on fixed table layout
Product: WebKit Reporter: Julien Chaffraix <jchaffraix>
Component: TablesAssignee: Julien Chaffraix <jchaffraix>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, ojan, robert, tony, webkit.review.bot
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Test case: the rows should be 100px wide, not 120px
none
Proposed fix. Take into account the box-sizing when computing the cell's size, refactored the code as we go.
none
Proposed fix (ct'd). Take into account the box-sizing when computing the cell's size, refactored the code as we go. none

Description Julien Chaffraix 2012-05-25 14:33:45 PDT
Created attachment 144148 [details]
Test case: the rows should be 100px wide, not 120px

During the review of bug 78412, box-sizing interaction was mentioned but unfortunately the patch went in without this being tested in any way.

This is unfortunate as box-sizing: border-box was the way to mitigate the web-compatibility issue from the original change: it make us correctly interpret width per CSS 2.1 but in an incompatible manner.
Comment 1 Julien Chaffraix 2012-05-25 16:39:00 PDT
Created attachment 144169 [details]
Proposed fix. Take into account the box-sizing when computing the cell's size, refactored the code as we go.
Comment 2 Julien Chaffraix 2012-05-25 16:51:22 PDT
Created attachment 144171 [details]
Proposed fix (ct'd). Take into account the box-sizing when computing the cell's size, refactored the code as we go.
Comment 3 Julien Chaffraix 2012-05-29 11:32:01 PDT
Comment on attachment 144171 [details]
Proposed fix (ct'd). Take into account the box-sizing when computing the cell's size, refactored the code as we go.

Thanks Tony!
Comment 4 WebKit Review Bot 2012-05-29 12:31:01 PDT
Comment on attachment 144171 [details]
Proposed fix (ct'd). Take into account the box-sizing when computing the cell's size, refactored the code as we go.

Clearing flags on attachment: 144171

Committed r118819: <http://trac.webkit.org/changeset/118819>
Comment 5 WebKit Review Bot 2012-05-29 12:31:07 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Andy Estes 2012-05-31 11:49:38 PDT
<rdar://problem/11567612>