1658814 - [wpt-sync] Sync PR 24982 - [weakref] WeakRef/FinalizationRegistry, cleanupSome testing

Status: RESOLVED FIXED

(Core :: JavaScript Engine, task, P4)

Description

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

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

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

Rick Waldron <[e-mail]> wrote:

[weakref] WeakRef/FinalizationRegistry, cleanupSome testing

@syg @leobalter This set of tests are built from the Test262 tests that allow for "maybe" garbage collection. Our harness utilities have been ported over, but with more obvious naming to make everything as clear as possible.

Comment 1

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

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

Comment 2

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

CI Results

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

Total 14 tests and 1 subtests

Status Summary

Firefox

OK : 14
FAIL : 30

Chrome

OK : 14
FAIL : 30

Safari

OK : 2
FAIL : 18
ERROR: 12

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/js/builtins/weakrefs/cleanup-prevented-with-reference.optional.any.worker.html
cleanupCallback has only one optional chance to be called for a GC that cleans up a registered target.: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/cleanup-prevented-with-reference.optional.any.html
cleanupCallback has only one optional chance to be called for a GC that cleans up a registered target.: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/cleanup-prevented-with-unregister.optional.any.html
Cleanup might be prevented with an unregister usage: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/cleanup-prevented-with-unregister.optional.any.worker.html
Cleanup might be prevented with an unregister usage: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/gc-has-one-chance-to-call-cleanupCallback.optional.any.html
cleanupCallback has only one optional chance to be called for a GC that cleans up a registered target.: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/gc-has-one-chance-to-call-cleanupCallback.optional.any.worker.html
cleanupCallback has only one optional chance to be called for a GC that cleans up a registered target.: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/holdings-multiple-values.optional.any.worker.html
Requires FinalizationRegistry.prototype.cleanupSome: FAIL (Chrome: FAIL, Safari: FAIL)
undefined as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
null as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
"" as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
{} as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
42 as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
true as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
false as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
Symbol(1) as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
/js/builtins/weakrefs/holdings-multiple-values.optional.any.html
Requires FinalizationRegistry.prototype.cleanupSome: FAIL (Chrome: FAIL, Safari: FAIL)
undefined as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
null as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
"" as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
{} as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
42 as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
true as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
false as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
Symbol(1) as registered holding value: FAIL (Chrome: FAIL, Safari: FAIL)
/js/builtins/weakrefs/reentrancy.optional.any.worker.html
cleanupCallback has only one optional chance to be called for a GC that cleans up a registered target.: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/reentrancy.optional.any.html
cleanupCallback has only one optional chance to be called for a GC that cleans up a registered target.: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/return-undefined-with-gc.optional.any.worker.html
Return undefined regardless the result of CleanupFinalizationRegistry: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/return-undefined-with-gc.optional.any.html
Return undefined regardless the result of CleanupFinalizationRegistry: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/unregister-cleaned-up-cell.optional.any.worker.html
Cannot unregister a cell that has been cleaned up: FAIL (Chrome: FAIL)
/js/builtins/weakrefs/unregister-cleaned-up-cell.optional.any.html
Cannot unregister a cell that has been cleaned up: FAIL (Chrome: FAIL)

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2190d8f0eb36
[wpt PR 24982] - [weakref] WeakRef/FinalizationRegistry, cleanupSome testing, a=testonly
https://hg.mozilla.org/integration/autoland/rev/c2e3694b8d0c
[wpt PR 24982] - Update wpt metadata, a=testonly

Comment 4

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/2190d8f0eb36
https://hg.mozilla.org/mozilla-central/rev/c2e3694b8d0c