tree 6c0b4eda7257d0b8fe0571d76f40128c932a2367
parent 48159f641f5208218a47d4dde463ac8f3e05e80a
author David Grogan <dgrogan@chromium.org> 1579056192 -0800
committer Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> 1579057095 -0800

[FlexNG] Fix min-width: auto for flex item tables

This patch only modifies the min-width: auto case for tables by skipping
the flex-specific behavior and just using the table's preferred minimum
width as calculated by legacy. We now match legacy when tables are flex
items.

But I think when the spec was updated in
https://github.com/w3c/csswg-drafts/commit/66241e4896d3e6300d8559080a0de4ee9a11fdcd,
legacy became incorrect in cases where min-width is specified. Compare
css/css-flexbox/table-as-item-fixed-min-width.html with
http://wpt.live/css/css-flexbox/table-as-item-auto-min-width.html . The
only difference is an additional min-width: 5px that should have no
effect but does, see https://i.imgur.com/yMxUdeI.png for how the new
test renders in legacy.

An older patchset (3) does what I think is the newly-correct behavior,
which also says that
http://wpt.live/css/css-flexbox/table-as-item-wide-content.html is
invalid -- the green square should be 500px wide.

Bug: 845235
Change-Id: Ica870fe3aa12fc1fbeb2388c0f2c69263c5e8860
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1995617
Commit-Queue: David Grogan <dgrogan@chromium.org>
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#731785}