1764451 - non262/Record/cross-realm.js test fails with --args=--more-compartments

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect, P1)

Description

[Tim Chevalier]

+++ This bug was initially created as a clone of Bug #1763874 +++

User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/96.0.4664.93 Safari/537.36

Steps to reproduce:

mach jstests Record/cross-realm --args=--more-compartments

Actual results:

## non262/Record/cross-realm.js: rc = -11, run time = 0.380883
Assertion failure: vp.isObject(), at /home/tjc/gecko-fork/js/src/vm/Compartment-inl.h:69
#01: JS::Compartment::wrap(JSContext*, JS::MutableHandle<JS::Value>)[/home/tjc/gecko-fork/obj-x64-debug/dist/bin/js +0x1b32596]
#02: js::CrossCompartmentWrapper::call(JSContext*, JS::Handle<JSObject*>, JS::CallArgs const&) const[/home/tjc/gecko-fork/obj-x64-debug/dist/bin/js +0x1e1e4be]
#03: ???[/home/tjc/gecko-fork/obj-x64-debug/dist/bin/js +0x1e257eb]

Expected results:

The test should pass. I have a patch coming shortly.

Comment 1

[Tim Chevalier]

Attachment: Bug 1764451 - Handle ExtendedPrimitives properly in JS::Compartment::wrap() r=jandem

Depends on D143310

Comment 2

[Tim Chevalier]

Ready to land D143551 -- try build at https://treeherder.mozilla.org/jobs?repo=try&revision=422af228fb0bf376d31db6d0fa24f51d123fe462 (it shows one test failure, but that seems spurious as the test passed when re-launched, there is a similar known issue (bug 1719221), and it seems unrelated to this change).

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2aaff54387ee
Handle ExtendedPrimitives properly in JS::Compartment::wrap() r=jandem

Comment 4

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/2aaff54387ee

Comment 5

[BugBot [:suhaib / :marco/ :calixte]]

Since the status are different for nightly and release, what's the status for beta?
For more information, please visit auto_nag documentation.