Closed Bug 1715592 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 29313 - [ReplacedNG] Fix max-content size of flex-item.

Categories

(Core :: Layout: Flexbox, task, P4)

Product:
Core
Component:
Layout: Flexbox
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
91 Branch
Tracking Flags:
Tracking Status
firefox91 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 29313 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/29313
Details from upstream follow.

b'Ian Kilpatrick <ikilpatrick@chromium.org>' wrote:

[ReplacedNG] Fix max-content size of flex-item.

Previously if we had an indefinite available inline-size, and an
aspect-ratio only replaced element (svg with viewBox), we'd "give up"
and use "zero".

This isn't what legacy does, instead it uses the "default natural size"
in the appropriate direction.

(Firefox uses the ICB size, but has bad invalidation).

This changes our behaviour to use the "default natural size" for this
case.

Bug: 1217941
Change-Id: Ia78d6c763c69b7c515fefacbf37b8184472fb4dd
Reviewed-on: https://chromium-review.googlesource.com/2950601
WPT-Export-Revision: f425209dc37e5064493e734d28de4036e6c8f3ab

Component: web-platform-tests → Layout: Flexbox
Product: Testing → Core
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/503e5c51d08c
[wpt PR 29313] - [ReplacedNG] Fix max-content size of flex-item., a=testonly
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/503e5c51d08c

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox91: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 91 Branch
You need to log in before you can comment on or make changes to this bug.