Closed Bug 1793000 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 36077 - [aspect-ratio] Fix legacy aspect-ratio code interference.

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
107 Branch
Tracking Status
firefox107 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Ian Kilpatrick <ikilpatrick@chromium.org> wrote:

[aspect-ratio] Fix legacy aspect-ratio code interference.

We use the LayoutBox::IntrinsicLogicalWidths call to retrieve previously
cached values within NG code.

This function call has some legacy logic within it messing up the
retrieval causing indeterminate state.

Fixed: 1335009
Change-Id: I6b2805707594b1d272add8db376def8f7dc73a44
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3917495
Commit-Queue: Ian Kilpatrick \<ikilpatrick@chromium.org>
Auto-Submit: Ian Kilpatrick \<ikilpatrick@chromium.org>
Reviewed-by: David Grogan \<dgrogan@chromium.org>
Commit-Queue: David Grogan \<dgrogan@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1051503}

Component: web-platform-tests → Layout
Product: Testing → Core
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0db4dbd0543f [wpt PR 36077] - [aspect-ratio] Fix legacy aspect-ratio code interference., a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 107 Branch
You need to log in before you can comment on or make changes to this bug.