Bug 221165 – In table with "width:max-content" and percent-width cells, first cell ends up infinitely-wide

• Status: RESOLVED
• Resolution: FIXED
• Component: Tables

[Daniel Holbert]

STR:
(1) Load this testcase:
https://jsfiddle.net/dholbert/yafp507j/
or this equivalent data URI:
data:text/html,<table border style="width:max-content"><td style="width:100%25">hi</td><td style="width:100%25">hi</td>

EXPECTED RESULTS:
No infinitely-sized content.

ACTUAL RESULTS:
The first table cell ends up infinitely-sized (or at least absurdly large).

Chrome gives ACTUAL RESULTS as well -- I suspect this is an issue that predates the blink/webkit fork. I filed https://bugs.chromium.org/p/chromium/issues/detail?id=1172498 to track this in Chrome.

Firefox gives EXPECTED RESULTS -- it just renders both cells as being content-sized.

[Radar WebKit Bug Importer]

<rdar://problem/74046127>

[Cameron McCormack (:heycam)]

Created attachment 424512 [details]
Patch

[Cameron McCormack (:heycam)]

Created attachment 424513 [details]
Patch

[alan]

Comment on attachment 424513 [details]
Patch

I am not too fond of the term "keyword" but I don't have a better candidate (unless we start renaming more functions here, but I don't think it's worth it)

[Cameron McCormack (:heycam)]

The spec term for these keywords in css-sizing-3 is "auto box size" keywords, so an alternative could be "computeAutoBoxLogicalWidths", but as you say there'd be a bunch of other renaming you'd want to do to stop using "intrinsic".

[EWS]

Committed r275462: <https://commits.webkit.org/r275462>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 424513 [details].