1981215 - [wpt-sync] Sync PR 54146 - [WebNN] Save NaN instead of 0 for MLNumber when initialized from NaN.

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

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

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

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

Wei Wang <[e-mail]> wrote:

[WebNN] Save NaN instead of 0 for MLNumber when initialized from NaN.

This CL saves NaN instead of 0 for MLNumber when initialized from
NaN. The behavior on different backends and hardwares are completely
inconsistent when NaN is used for min_value or max_value of clamp.
So we normalize it with minValue:NaN coerced to -Infinity and
maxValue:NaN be coerced to Infinity.

Bug: 335909582
Change-Id: I4c7467b573010f7804438dc1d2993e45594b0708
Reviewed-on: https://chromium-review.googlesource.com/6805191
WPT-Export-Revision: 9171b0aa0deae83b860f9131f823765c72d7e12b

Comment 1

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

PR 54146 applied with additional changes from upstream: 3a758d9691dca742534c38429a3d7b704e4fb75d

Comment 2

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

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=c18c8036f267fc87e9eb09c0375cc2a1bc6741d4

Comment 3

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

Attachment: Notable wpt changes

# CI Results

Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 387 tests

## Status Summary

### Firefox
`OK`  : 6
`FAIL`: 6

### Chrome
`OK`  : 6
`PASS`: 180
`FAIL`: 45

### Safari
`OK`  : 6
`FAIL`: 6

## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=c18c8036f267fc87e9eb09c0375cc2a1bc6741d4)
[GitHub PR Head](https://wpt.fyi/results/?sha=eea56481cf8fc083c229f47c46d1d34ed337cc3b&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=eea56481cf8fc083c229f47c46d1d34ed337cc3b&label=pr_base)

## Details

(See attachment for full changes)

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/c9bb4a74aa8d
https://hg.mozilla.org/integration/autoland/rev/611ca8ba2fb7
[wpt PR 54146] - [WebNN] Save NaN instead of 0 for MLNumber when initialized from NaN., a=testonly

Comment 5

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/611ca8ba2fb7