Closed Bug 1757450 Opened 9 months ago Closed 9 months ago

[wpt-sync] Sync PR 33004 - [layout] Resolve margins using zero %-size for content contribution.

Categories

(Core :: Layout, task, P4)

task

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Ian Kilpatrick <ikilpatrick@chromium.org> wrote:

[layout] Resolve margins using zero %-size for content contribution.

Previously for a box's content contribution we'd only resolve fixed
margins (e.g. margin-left: 10px).

We'd treat things like "margin-left: calc(10% + 100px)" as indefinite.
This patch changes the logic to resolve anything with a percent
against zero.

Bug: 823066
Change-Id: I191f153d288319e7af0db09bb1be269dd60cc325

Reviewed-on: https://chromium-review.googlesource.com/3481537
WPT-Export-Revision: bb1d4e95cdb3e6f8d1c9ca3e5457f72b49f4f75f

Component: web-platform-tests → Layout
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream error]
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eef67c9a465b
[wpt PR 33004] - [layout] Resolve margins using zero %-size for content contribution., a=testonly
Test result changes from PR not available.
Regressions: 1760566
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.