Closed
Bug 1429200
Opened 7 years ago
Closed 6 years ago
split service worker DOM objects into inner/outer objects
Categories
(Core :: DOM: Service Workers, enhancement, P2)
Core
DOM: Service Workers
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.
Assignee | ||
Comment 1•7 years ago
|
||
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.
Updated•7 years ago
|
Priority: -- → P2
Comment 2•7 years ago
|
||
we're targeting this for 60 right?
Assignee | ||
Comment 3•7 years ago
|
||
Yes.
Assignee | ||
Updated•7 years ago
|
Attachment #8941227 -
Attachment is obsolete: true
Assignee | ||
Comment 4•7 years ago
|
||
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
Assignee | ||
Comment 5•6 years ago
|
||
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.
Description
•