Closed Bug 1429200 Opened 6 years ago Closed 6 years ago

split service worker DOM objects into inner/outer objects

Categories

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

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: bkelly, Assigned: bkelly)

References

(Blocks 1 open bug)

Details

Attachments

(1 obsolete file)

In order to move ServiceWorkerManager to the parent process we need to make the various service worker DOM object operate remotely over IPC.  As a step towards that end I'm going to split the current DOM objects into inner and outer objects.  Once this is done we can have a pref to select whether we use local or remote inner objects.
WIP patch.  A couple things I had to comment out to get it to compile while I think about the best way to structure things.
Depends on: 1429486
Priority: -- → P2
we're targeting this for 60 right?
Yes.
Depends on: 1433505
Depends on: 1433545
Depends on: 1433568
Attachment #8941227 - Attachment is obsolete: true
After thinking about this more, I think this bug is going to be end up as a meta bug.  Changes will be made incrementally in dependent bugs.  Overall the plan is:

1. Make the DOM objects like ServiceWorker and ServiceWorkerRegstration work based on the IPC-safe descriptor types. They will attach to an active inner object which roughly corresponds to our current Info classes.
2. The active inner objects need to be completely asynchronous and avoid direct access to child-process stuff.
3. Implement a separate inner objects that attach directly to local Info class objects or use PBackground actors.  Selection based on a pref.

A nice side effect is that this work is also largely what we need to do to solve bug 1113522.
Blocks: 1113522
Depends on: 1434342
Depends on: 1434701
Depends on: 1436812
Depends on: 1455695
Depends on: 1456981
Depends on: 1456986
Blocks: 1459209
All the dependents are fixed.  This is done.
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: