Closed Bug 1850619 Opened 1 year ago Closed 1 year ago

[wpt-sync] Sync PR 41702 - CIS: Switch intrinsic length type to Length instead of LayoutUnit.

Categories

(Core :: Layout, task, P4)

Product:
Core
Component:
Layout
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
119 Branch
Tracking Flags:
Tracking Status
firefox119 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Vladimir Levin <vmpstr@chromium.org> wrote:

CIS: Switch intrinsic length type to Length instead of LayoutUnit.

This regressed when we introduced contain-intrinsic-size auto none,
since I switched to using LayoutUnit. However, there is a difference
in computed value vs used value when using a length. The wpt.fyi tests
started failing, but because of our testrunner's viewport size, local
tests kept passing. I've added another line to the tests to ensure they
would fail locally as well (without this patch).

R=andruud@chromium.org

Fixed: 1476874
Change-Id: I08261d014daee7d57685af82606769803ac2f6be
Reviewed-on: https://chromium-review.googlesource.com/4822945
WPT-Export-Revision: f464a55a149bc102aebcac0f4de80730dab52472

Component: web-platform-tests → Layout
Product: Testing → Core
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cc5ac000320f [wpt PR 41702] - CIS: Switch intrinsic length type to Length instead of LayoutUnit., a=testonly

https://hg.mozilla.org/mozilla-central/rev/cc5ac000320f

Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox119: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
You need to log in before you can comment on or make changes to this bug.