115039 2013-04-23 07:50:01 -0700 [CSSRegions]Flex items auto-height regions have wrong height 2013-07-31 13:08:42 -0700 1 1 1 Unclassified WebKit CSS 528+ (Nightly build) Unspecified Unspecified RESOLVED INVALID AdobeTracked P2 Normal --- 118057 57312 1 mihnea webkit-unassigned badea mibalan WebkitBugTracker oldest_to_newest 879842 0 199234 mihnea 2013-04-23 07:50:01 -0700 Created attachment 199234 Test case If an auto-height region is a flex item, its height is not computed properly as it does not take the flowed content height into account (see attached sample). 879859 1 mihnea 2013-04-23 07:58:32 -0700 In the second pass of layout for auto-height regions, the flexbox container clears the overrideLogicalHeight for the autoHeight region before the overrideLogicalHeight (which stores the computed autoHeight value after the first phase) is transfered into the regions height. 880178 2 199334 stearns 2013-04-23 16:46:49 -0700 Created attachment 199334 test file 2 Here's another test file. In this case the region in the flex container is taking on the height of the container, but the second region isn't sizing correctly (there should be a black line around the second region's content - what you're seeing is overflow). If you take the border off the regions, the overflow in the second region disappears (perhaps this is a separate bug?) 912700 3 badea 2013-07-30 01:28:55 -0700 The issue from the first test case has been fixed in #118057. (In reply to comment #2) > Created an attachment (id=199334) [details] > test file 2 > > Here's another test file. In this case the region in the flex container is taking on the height of the container, but the second region isn't sizing correctly (there should be a black line around the second region's content - what you're seeing is overflow). If you take the border off the regions, the overflow in the second region disappears (perhaps this is a separate bug?) The fact that the second region does not increases its size for fitting the content proved to be the correct behaviour according to the spec. The rfcb height resolution with auto-height regions inside flexboxes roughly works as follows: 1. layout document using height=0 for auto-height regions. This includes a full flexbox layout. 2. Layout flow thread content and compute the height of auto-height regions ignoring any constraints set by the flex container. 3. layout document using the computed heights from step 2, this time honoring the constraints set by the flex container. What this means is that, for test 2, the first region will compute a height equal to the flowthread's height and the second region will compute a height equal to zero (in step 2). In step 3 the flex container will apply constraints on the first region shrinking it and causing overflow. The overflow not being displayed unless the region with zero height has a border is another bug, indeed. 913469 4 mibalan 2013-07-31 13:08:42 -0700 Closing this one as per the above discussion and the conclusion that the current behavior is spec-compliant. For the zero-height region issue, bug 98265 is already logged. 199234 2013-04-23 07:50:01 -0700 2013-04-23 07:50:01 -0700 Test case region-flex-autoheight.html text/html 587 mihnea PCFkb2N0eXBlIGh0bWw+CjxodG1sPgo8aGVhZD4KICA8c3R5bGU+CiAgICBhcnRpY2xlIHsKICAg ICAgLXdlYmtpdC1mbG93LWludG86IGFydGljbGU7CiAgICB9CgogICAgLmNvbnRhaW5lcnsKICAg ICAgd2lkdGg6IDIwJTsKICAgICAgZGlzcGxheTogLXdlYmtpdC1mbGV4OwogICAgICBib3JkZXI6 IDVweCBzb2xpZCBibHVlOwogICAgfQoKICAgIC5jZWxsIHsKICAgICAgLXdlYmtpdC1mbG93LWZy b206IGFydGljbGU7CiAgICAgIC13ZWJraXQtZmxleC1ncm93OiAxOwogICAgICAtd2Via2l0LWZs ZXgtYmFzaXM6IDEwMCU7CiAgICAgIGhlaWdodDogYXV0bzsKICAgICAgYmFja2dyb3VuZDogbGlt ZTsKICAgICAgYm9yZGVyOiA1cHggc29saWQgZ3JlZW47CiAgICB9CiAgPC9zdHlsZT4KPC9oZWFk Pgo8Ym9keT4KICA8YXJ0aWNsZT4KICAgIDxwPkxvcmVtIGlwc3VtIGRvbG9yIHNpdCBhbWV0LCBj b25zZWN0ZXR1ciBhZGlwaXNpY2luZyBlbGl0LjwvcD4KICA8L2FydGljbGU+CgogIDxkaXYgY2xh c3M9ImNvbnRhaW5lciI+CiAgICA8ZGl2IGNsYXNzPSJjZWxsIj4KICAgIDwvZGl2PgogIDwvZGl2 Pgo8L2JvZHk+CjwvaHRtbD4= 199334 2013-04-23 16:46:49 -0700 2013-04-23 16:46:49 -0700 test file 2 flex-fragmenting.html text/html 990 stearns PCFET0NUWVBFIGh0bWw+CjxodG1sPgo8aGVhZD4KICA8bWV0YSBodHRwLWVxdWl2PSJDb250ZW50 LVR5cGUiIGNvbnRlbnQ9InRleHQvaHRtbDsgY2hhcnNldD11dGYtOCI+CiAgPHRpdGxlPkZsZXgt cmVnaW9uIHRlc3Q8L3RpdGxlPgogIDxzdHlsZT4KICAgIGFydGljbGUgewogICAgICAtd2Via2l0 LWZsb3ctaW50bzogYXJ0aWNsZS1mbG93OwogICAgfQogICAgLnJlZ2lvbiB7CiAgICAgIC13ZWJr aXQtZmxvdy1mcm9tOiBhcnRpY2xlLWZsb3c7CiAgICAgIGJvcmRlcjogdGhpbiBibGFjayBzb2xp ZDsKICAgIH0KICAgIC5mbGV4LWNvbnRhaW5lciB7CiAgICAgIGRpc3BsYXk6IC13ZWJraXQtZmxl eDsKICAgICAgaGVpZ2h0OiA1ZW07CiAgICAgIHdpZHRoOiAzMDBweDsKICAgIH0KICA8L3N0eWxl Pgo8L2hlYWQ+Cjxib2R5IGxhbmc9ImVuIj4KICA8ZGl2IGNsYXNzPSJmbGV4LWNvbnRhaW5lciI+ CiAgICA8ZGl2IGNsYXNzPSJyZWdpb24iPjwvZGl2PgogIDwvZGl2PgogIDxkaXYgY2xhc3M9InJl Z2lvbiI+PC9kaXY+CgogIDxhcnRpY2xlPgogIDxwPlRodXMgbXVzaW5nLCBoZSB0b29rIGhpcyBz dGFuZCBhdCB0aGUgZW50cmFuY2Ugb2YgdGhlIGdhdGUgYW5kIHdhaXRlZCB1bnRpbCB0aGUgcGVy c29uYWdlIHNvIHNpbmd1bGFybHkgYW5ub3VuY2VkIHNob3VsZCBtYWtlIGhpcyBhcHBlYXJhbmNl LiBBZnRlciBhbiBpbnN0YW50IG9yIHR3byBoZSBiZWhlbGQgdGhlIGZpZ3VyZSBvZiBhIGxlYW4g bWFuLCBvZiB1bndob2xlc29tZSBsb29rLCB3aXRoIGdsaXR0ZXJpbmcgZXllcyBhbmQgbG9uZyBi bGFjayBoYWlyLCB3aG8gc2VlbWVkIHRvIGltaXRhdGUgdGhlIG1vdGlvbiBvZiBhIHNuYWtlOyBm b3IsIGluc3RlYWQgb2Ygd2Fsa2luZyBzdHJhaWdodCBmb3J3YXJkIHdpdGggb3BlbiBmcm9udCwg aGUgdW5kdWxhdGVkIGFsb25nIHRoZSBwYXZlbWVudCBpbiBhIGN1cnZlZCBsaW5lLgogIDwvYXJ0 aWNsZT4KPC9ib2R5Pgo8L2h0bWw+