Closed Bug 1583987 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 19309 - Do not simplify calculation across math functions

Categories

(Core :: CSS Parsing and Computation, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox71 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Xiaocheng Hu <xiaochengh@chromium.org> wrote:

Do not simplify calculation across math functions

The existing code is too aggressive in simplifying calc() expressions
that, math functions (min/max etc) can be simplified away when they
can be directly resolved. This doesn't match the spec (*), and is fixed
in this patch.

(*) https://drafts.csswg.org/css-values-4/#math-function-simplify-a-calculation

Bug: 825895
Change-Id: Id5796299da50e40b09bc38bf0271c8f5cd7897de
Reviewed-on: https://chromium-review.googlesource.com/1825325
WPT-Export-Revision: 9608f2654baa344f127dd8253ef050d6f3dd20b4

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Ran 1 tests and 10 subtests
OK     : 1
FAIL   : 10

New tests that have failures or other problems:
/css/css-values/minmax-angle-serialize.html
    e.style['transform'] = "rotate(calc(max(1deg) + max(2deg)))" should set the property value: FAIL
    e.style['transform'] = "rotate(calc(min(1deg) + min(2deg)))" should set the property value: FAIL
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5b5b09da541f
[wpt PR 19309] - Do not simplify calculation across math functions, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/99b2e3d332f5
[wpt PR 19309] - Update wpt metadata, a=testonly
You need to log in before you can comment on or make changes to this bug.