Closed Bug 1779603 Opened 2 years ago Closed 2 years ago

Modify xorigin Mochitest variant to disable Always Partition Storage permission

Categories

(Core :: Privacy: Anti-Tracking, task)

task

Tracking

()

RESOLVED FIXED
104 Branch
Tracking Status
firefox104 --- fixed

People

(Reporter: bvandersloot, Assigned: bvandersloot)

References

Details

Attachments

(1 file)

This is a temporary measure that will allow us to experiment with Always Partition Storage without changing all tests to follow the new behavior. We changed all other tests to be compatible, but many xorigin mochitests are not only because of their use of BroadcastChannels or other similar APIs that assume shared storage between embedded iframes and top level contexts from the same origin.

Assignee: nobody → bvandersloot
Status: NEW → ASSIGNED
Pushed by bvandersloot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e3c8cd6f00b6
Disable Always Partition Storage on xorigin mochitests, r=jmaher

Backed out for causing mochitest failures on browser_partitionedConsoleMessage.js

Flags: needinfo?(bvandersloot)

There is a race condition introduced in this test that was not catching on my local machine (and seemed to eventually pass on the try run with enough retries). I'm going to fix it.

It's actually a one of those fun "create the promise to wait for the observable behavior after the action that causes it" types that usually just fail. But the console is just slow enough on my machine to reliably work. This was a fun one :)

Flags: needinfo?(bvandersloot)
Pushed by bvandersloot@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/cd3007f4be5d
Disable Always Partition Storage on xorigin mochitests, r=jmaher
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 104 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: