Bug 56174 - Table cells with dynamically added percentage height descendants do not grow in size if the table has already been laid out.
Summary: Table cells with dynamically added percentage height descendants do not grow ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Andy Estes
URL:
Keywords: HasReduction, InRadar
Depends on:
Blocks:
 
Reported: 2011-03-11 01:15 PST by Andy Estes
Modified: 2011-03-11 13:21 PST (History)
5 users (show)

See Also:


Attachments
Test case (996 bytes, text/html)
2011-03-11 01:18 PST, Andy Estes
no flags Details
Patch (7.87 KB, patch)
2011-03-11 01:48 PST, Andy Estes
hyatt: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Estes 2011-03-11 01:15:39 PST
Table cells with dynamically added percentage height descendants do not grow in size if the table has already been laid out.
Comment 1 Andy Estes 2011-03-11 01:18:25 PST
Created attachment 85444 [details]
Test case
Comment 2 Andy Estes 2011-03-11 01:18:40 PST
<rdar://problem/7863676>
Comment 3 Andy Estes 2011-03-11 01:19:59 PST
Here is why I think this happens:

Table cells ignore their specified size and collapse to fit their children. When the child div is added in the same run loop iteration as the table itself, the TD is sized appropriately (100% of the 100px TD). When the child div isn't present when the table is first laid out, the TD ignores its 100px sizing and collapses down. Then, when the child div is added in a separate run loop iteration, it is sized to be 100% of the collapsed TD instead of 100% of the TD's specified height of 100px.
Comment 4 Andy Estes 2011-03-11 01:48:13 PST
Created attachment 85448 [details]
Patch
Comment 5 Dave Hyatt 2011-03-11 10:53:52 PST
Comment on attachment 85448 [details]
Patch

r=me
Comment 6 Andy Estes 2011-03-11 13:21:40 PST
Committed r80878: <http://trac.webkit.org/changeset/80878>