Closed Bug 1556046 Opened 1 year ago Closed 1 year ago

Add tests to ensure separate tabs with partitioning get partitioned storage

Categories

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

task
Not set

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: ehsan, Assigned: baku)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Right now the partitionedstorage_head.js test suite verifies that the storage/communication APIs work correctly before/after partitioning. But we also need tests to ensure that the partitioning itself correctly separates data from different top-level origins. It would be nice to extend PartitionedStorageHelper with the ability to verify that storage in tabs from separate top-level origins is actually separate.

I propose the following:

  • Add a new function to PartitionedStorageHelper call runPartitioningTest() as the entry point for the new tests.
  • In the new tests, open two top-level tabs from two different origins, and verify that the third-party storage in each tab lives in its own universe, and when they both call the storage access API they're moved to the same universe.
  • Extend each of the browser_partitioned* tests with calls to this new entry point function.
  • It could be that we come across new bugs in some of these tests. Where needed, comment out tests that do not pass yet and file bugs on enabling them later on to be able to land this test before all of those bugs are fixed.

Andrea, do you feel like taking this? Thanks!

Flags: needinfo?(amarchesini)

Bugbug thinks this bug is a task, but please change it back in case of error.

Type: defect → task
Assignee: nobody → amarchesini
Flags: needinfo?(amarchesini)
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/05811533a472
Add tests to ensure separate tabs with partitioning get partitioned storage, r=Ehsan
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
You need to log in before you can comment on or make changes to this bug.