1820412 - Comp funcs (min/max/clamp) should handle NaN contagiously

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Oliver Medhurst [:canadahonk]]

NaN should be contagious (always the result) in comp funcs (min/max/clamp).

WPT tests which check for this behavior:

• https://wpt.fyi/results/css/css-values/calc-infinity-nan-computed.html
• https://wpt.fyi/results/css/css-values/calc-infinity-nan-serialize-angle.html
• https://wpt.fyi/results/css/css-values/calc-infinity-nan-serialize-length.html
• https://wpt.fyi/results/css/css-values/calc-infinity-nan-serialize-time.html

(possibly more subtests in various others)

Comment 1

[Oliver Medhurst [:canadahonk]]

Attachment: Bug 1820412 - CSS comp funcs should handle NaN contagiously

If NaN is given as any input to CSS comp funcs (min/max/clamp), it
should return NaN.

Adjusted WPT test expectations, 4 newly pass :tada:
(and more when other types get NaN support later)

Depends on D171658

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c1e40ba0719d
CSS comp funcs should handle NaN contagiously r=emilio

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/39240 for changes under testing/web-platform/tests

Comment 4

[Narcis Beleuzu [:NarcisB]]

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

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot