Closed
Bug 1941765
Opened 11 months ago
Closed 11 months ago
Perma 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 cookieStore when Gecko 136 merges to beta on 2025-02-03
Categories
(Core :: DOM: Service Workers, defect)
Core
DOM: Service Workers
Tracking
()
VERIFIED
FIXED
136 Branch
| Tracking | Status | |
|---|---|---|
| firefox-esr115 | --- | unaffected |
| firefox-esr128 | --- | unaffected |
| firefox134 | --- | unaffected |
| firefox135 | --- | unaffected |
| firefox136 | + | verified |
People
(Reporter: abutkovits, Assigned: baku)
References
(Regression)
Details
(Keywords: regression)
Attachments
(2 files)
[task 2025-01-15T11:29:07.284Z] 11:29:07 INFO - Buffered messages finished
[task 2025-01-15T11:29:07.285Z] 11:29:07 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 cookieStore to all webpages as a property on self? 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 2025-01-15T11:29:07.285Z] 11:29:07 INFO - SimpleTest.ok@SimpleTest/SimpleTest.js:426:16
[task 2025-01-15T11:29:07.285Z] 11:29:07 INFO - setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2025-01-15T11:29:07.285Z] 11:29:07 INFO - EventHandlerNonNull*setupSW@dom/serviceworkers/test/test_serviceworker_interfaces.html:19:5
[task 2025-01-15T11:29:07.285Z] 11:29:07 INFO - promise callback*runTest@dom/serviceworkers/test/test_serviceworker_interfaces.html:86:8
[task 2025-01-15T11:29:07.285Z] 11:29:07 INFO - Async*onload@dom/serviceworkers/test/test_serviceworker_interfaces.html:96:19
[task 2025-01-15T11:29:07.285Z] 11:29:07 INFO - EventHandlerNonNull*@dom/serviceworkers/test/test_serviceworker_interfaces.html:90:3
[task 2025-01-15T11:29:07.287Z] 11:29:07 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 AbortController to all webpages as a property on self? 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 2025-01-15T11:29:07.287Z] 11:29:07 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 AbortSignal to all webpages as a property on self? 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 2025-01-15T11:29:07.287Z] 11:29:07 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 Blob to all webpages as a property on self? 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 2025-01-15T11:29:07.287Z] 11:29:07 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 BroadcastChannel to all webpages as a property on self? 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 2025-01-15T11:29:07.288Z] 11:29:07 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 ByteLengthQueuingStrategy to all webpages as a property on self? 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 2025-01-15T11:29:07.288Z] 11:29:07 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 Cache to all webpages as a property on self? 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 2025-01-15T11:29:07.289Z] 11:29:07 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 CacheStorage to all webpages as a property on self? 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 2025-01-15T11:29:07.290Z] 11:29:07 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 CanvasGradient to all webpages as a property on self? 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 2025-01-15T11:29:07.290Z] 11:29:07 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 CanvasPattern to all webpages as a property on self? 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 2025-01-15T11:29:07.291Z] 11:29:07 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 Client to all webpages as a property on self? 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 2025-01-15T11:29:07.294Z] 11:29:07 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 Clients to all webpages as a property on self? 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 2025-01-15T11:29:07.294Z] 11:29:07 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 Event to all webpages as a property on self? 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 2025-01-15T11:29:07.294Z] 11:29:07 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 CloseEvent to all webpages as a property on self? 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 2025-01-15T11:29:07.294Z] 11:29:07 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 CompressionStream to all webpages as a property on self? 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 2025-01-15T11:29:07.295Z] 11:29:07 INFO - Not taking screenshot here: see the one that was previously logged
[task 2025-01-15T11:29:07.295Z] 11:29:07 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 CookieChangeEvent to all webpages as a property on self? 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 2025-01-15T11:29:07.295Z] 11:29:07 INFO - SimpleTest.ok@SimpleTest/SimpleTest.js:426:16
[task 2025-01-15T11:29:07.295Z] 11:29:07 INFO - setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2025-01-15T11:29:07.295Z] 11:29:07 INFO - EventHandlerNonNull*setupSW@dom/serviceworkers/test/test_serviceworker_interfaces.html:19:5
[task 2025-01-15T11:29:07.295Z] 11:29:07 INFO - promise callback*runTest@dom/serviceworkers/test/test_serviceworker_interfaces.html:86:8
[task 2025-01-15T11:29:07.295Z] 11:29:07 INFO - Async*onload@dom/serviceworkers/test/test_serviceworker_interfaces.html:96:19
[task 2025-01-15T11:29:07.295Z] 11:29:07 INFO - EventHandlerNonNull*@dom/serviceworkers/test/test_serviceworker_interfaces.html:90:3
[task 2025-01-15T11:29:07.295Z] 11:29:07 INFO - Not taking screenshot here: see the one that was previously logged
[task 2025-01-15T11:29:07.296Z] 11:29:07 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 CookieStore to all webpages as a property on self? 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 2025-01-15T11:29:07.296Z] 11:29:07 INFO - SimpleTest.ok@SimpleTest/SimpleTest.js:426:16
[task 2025-01-15T11:29:07.296Z] 11:29:07 INFO - setupSW/window.onmessage@dom/serviceworkers/test/test_serviceworker_interfaces.html:31:11
[task 2025-01-15T11:29:07.296Z] 11:29:07 INFO - EventHandlerNonNull*setupSW@dom/serviceworkers/test/test_serviceworker_interfaces.html:19:5
[task 2025-01-15T11:29:07.297Z] 11:29:07 INFO - promise callback*runTest@dom/serviceworkers/test/test_serviceworker_interfaces.html:86:8
[task 2025-01-15T11:29:07.299Z] 11:29:07 INFO - Async*onload@dom/serviceworkers/test/test_serviceworker_interfaces.html:96:19
[task 2025-01-15T11:29:07.299Z] 11:29:07 INFO - EventHandlerNonNull*@dom/serviceworkers/test/test_serviceworker_interfaces.html:90:3 ```
Flags: needinfo?(amarchesini)
Updated•11 months ago
|
| Assignee | ||
Updated•11 months ago
|
Flags: needinfo?(amarchesini)
| Assignee | ||
Comment 1•11 months ago
|
||
Updated•11 months ago
|
Assignee: nobody → amarchesini
Status: NEW → ASSIGNED
Updated•11 months ago
|
Attachment #9459627 -
Attachment description: Bug 1941765 - Expose CoookieStore API to early beta - test fix, r?valentin → Bug 1941765 - Expose CookieStore API to early beta - test fix, r?valentin
| Reporter | ||
Comment 2•11 months ago
|
||
Could this also be caused by the same issue?
https://treeherder.mozilla.org/logviewer?job_id=490224228&repo=try&lineNumber=20448
Flags: needinfo?(amarchesini)
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8bfeb21c4f7a
Expose CookieStore API to early beta - test fix, r=valentin,webidl,saschanaz
Comment 4•11 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
Updated•11 months ago
|
Flags: needinfo?(amarchesini)
Comment 5•11 months ago
|
||
Hi! This failure still remains as it was already mentioned in Comment 2. Could you please take a look at this?
Thank you!
Status: RESOLVED → REOPENED
Flags: needinfo?(amarchesini)
Resolution: FIXED → ---
Target Milestone: 136 Branch → ---
| Assignee | ||
Comment 6•11 months ago
|
||
Comment 7•11 months ago
|
||
Patch is working in today's central as beta simulation.
| Comment hidden (Intermittent Failures Robot) |
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4508d04f5408
Disable cookie-store API for service-worker tests, r=valentin
| Assignee | ||
Updated•11 months ago
|
Flags: needinfo?(amarchesini)
Comment 10•11 months ago
|
||
I'm confused. What has that latest patch to do with dom/serviceworkers/test/test_serviceworker_interfaces.html ?
Flags: needinfo?(amarchesini)
| Assignee | ||
Comment 11•11 months ago
|
||
After reading comment 7, can we close this bug?
Flags: needinfo?(amarchesini) → needinfo?(sstanca)
Updated•11 months ago
|
Status: REOPENED → RESOLVED
Closed: 11 months ago → 11 months ago
Flags: needinfo?(sstanca)
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
Comment 12•11 months ago
|
||
| bugherder | ||
Comment 13•11 months ago
|
||
Verified fixed in today’s beta simulation push.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•