Closed Bug 1337439 Opened 8 years ago Closed 7 years ago

write multi-e10s Clients API test

Categories

(Core :: DOM: Service Workers, defect, P2)

Product:
Core
Component:
DOM: Service Workers
Type:
defect

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: bkelly, Assigned: bkelly)

References

Details

We should have a test that guarantees windows are opened in separate processes and then verifies the Clients API works. The test should probably exercise the `includeUncontrolled` option in `clients.matchAll()`. We do this in a WPT test currently: https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/service-workers/service-worker/clients-matchall-include-uncontrolled.https.html We could write a browser chrome or mochitest which does something similar with windows forced to be in separate tabs.
I suppose this is needed for multi-e10s, telling from the bug dependency, so mark as P1. Feel free to change it if I misunderstand something, Ben.
Priority: -- → P1
Priority: P1 → P2
I'm going to flip the dependencies around here. I'll look at writing this test after adding support in bug 1293277.
Assignee: nobody → bkelly
No longer blocks: 1293277
Status: NEW → ASSIGNED
Depends on: 1293277
I think we are going to have more than enough test coverage through existing tests.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.