[Fission] Make BrowserChild ready for Fission
Categories
(Toolkit :: General, enhancement, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox69 | --- | fixed |
People
(Reporter: Felipe, Assigned: mconley)
References
Details
Attachments
(2 files, 1 obsolete file)
Make the code in BrowserChild.jsm ready to work with out-of-process iframes
Reporter | ||
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Hey jdai, I have a patch up here that'll port one of our old-school actors to JS Window Actors, but I've realized that that's not going to be a full Fission-compatible port in this case.
BrowserParent / BrowserChild are responsible for determining whether or not a tab can be unloaded, or if a page is blocking that unload via the beforeunload event.
For multiple frames across multiple processes, I think we're going to need a way of broadcasting a message to all of the subframes, and then getting a result from each.
Is that broadcast capability planned?
Reporter | ||
Comment 3•6 years ago
|
||
hey, the patch is missing the BrowserParent.jsm file!
fwiw, the browsingContext in the parent side has a getChildren() function that can be used to enumerate all children BCs* and then use them to call .getActor().. At least, that was the original plan for broadcasting.. I don't know if some built-in functionality is also planned.
- note that one call to getChildren is not enough.. it requires tree traversal as the items returned could themselves have children.
Comment 4•6 years ago
|
||
Since :Felipe already answered your question, I'm going to cancel the needinfo request. Feel free to open a bug, if you need any functionality. I'm willing to help. :)
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Assignee | ||
Comment 5•6 years ago
|
||
Updated•6 years ago
|
Assignee | ||
Comment 6•6 years ago
|
||
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/2d6ba0372205
https://hg.mozilla.org/mozilla-central/rev/574775878141
Description
•