Closed Bug 1761257 Opened 2 years ago Closed 2 years ago

Perma dom/workers/test/test_worker_interfaces.html | false: If this is failing: DANGER, are you sure you want to expose the new interface Tag to all webpages as a property of WebAssembly? Do not make a change... when Gecko 100 merges to Beta on 2022-04-04

Categories

(Core :: DOM: Workers, defect)

defect

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox98 --- unaffected
firefox99 --- unaffected
firefox100 + verified

People

(Reporter: noriszfay, Assigned: rhunt)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Central-as-beta simulation
How to run these simulations

Failure log:

[task 2022-03-24T12:09:57.661Z] 12:09:57     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | true: If this is failing: DANGER, are you sure you want to expose the new interface LinkError to all webpages as a property on WebAssembly? Do not make a change to this file without a  review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) 
[task 2022-03-24T12:09:57.662Z] 12:09:57     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | true: If this is failing: DANGER, are you sure you want to expose the new interface RuntimeError to all webpages as a property on WebAssembly? Do not make a change to this file without a  review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) 
[task 2022-03-24T12:09:57.662Z] 12:09:57     INFO - Buffered messages finished
[task 2022-03-24T12:09:57.663Z] 12:09:57     INFO - TEST-UNEXPECTED-FAIL | dom/serviceworkers/test/test_serviceworker_interfaces.html | false: If this is failing: DANGER, are you sure you want to expose the new interface Tag to all webpages as a property on WebAssembly? Do not make a change to this file without a  review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) 
[task 2022-03-24T12:09:57.663Z] 12:09:57     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:417:16
[task 2022-03-24T12:09:57.663Z] 12:09:57     INFO -     setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2022-03-24T12:09:57.664Z] 12:09:57     INFO - Not taking screenshot here: see the one that was previously logged
[task 2022-03-24T12:09:57.664Z] 12:09:57     INFO - TEST-UNEXPECTED-FAIL | dom/serviceworkers/test/test_serviceworker_interfaces.html | false: If this is failing: DANGER, are you sure you want to expose the new interface Exception to all webpages as a property on WebAssembly? Do not make a change to this file without a  review from a DOM peer for that specific change!!! (or a JS peer for changes to ecmaGlobals) 
[task 2022-03-24T12:09:57.664Z] 12:09:57     INFO -     SimpleTest.ok@SimpleTest/SimpleTest.js:417:16
[task 2022-03-24T12:09:57.664Z] 12:09:57     INFO -     setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2022-03-24T12:09:57.665Z] 12:09:57     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | true: Function should  NOT be defined on WebAssembly 
[task 2022-03-24T12:09:57.666Z] 12:09:57     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | 0 === 0: The following interface(s) are not enumerated:  
[task 2022-03-24T12:09:57.666Z] 12:09:57     INFO - TEST-PASS | dom/serviceworkers/test/test_serviceworker_interfaces.html | The service worker should be unregistered successfully 
[task 2022-03-24T12:09:57.666Z] 12:09:57     INFO - GECKO(7000) | MEMORY STAT | vsize 19405595MB | vsizeMaxContiguous 68314409MB | residentFast 632MB
[task 2022-03-24T12:09:57.667Z] 12:09:57     INFO - TEST-OK | dom/serviceworkers/test/test_serviceworker_interfaces.html | took 785ms
[task 2022-03-24T12:09:57.667Z] 12:09:57     INFO - TEST-START | dom/serviceworkers/test/test_serviceworker_not_sharedworker.html

Hello Emilio, could you take a look at this and check if this started from Bug 1759962 ? If not, could you please redirect it to the right person? Thank you!

Flags: needinfo?(emilio)

Ryan, Wasm exceptions is your thing right? :)

Flags: needinfo?(emilio) → needinfo?(rhunt)
Regressed by: 1759217

Yup, will fix. Thanks!

Assignee: nobody → rhunt
Flags: needinfo?(rhunt)
Has Regression Range: --- → yes

The patch seems to be working as of today's beta sim

Pushed by rhunt@eqrion.net:
https://hg.mozilla.org/integration/autoland/rev/f05db0a3e06e
wasm: Update interface tests for exception handling shipping. r=yury
Severity: -- → S2
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: