Use BrowsingContext (and IPC) in place of GetInProcessParent usage in nsGlobalWindowOuter::FocusOuter
Categories
(Core :: DOM: Core & HTML, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox82 | --- | fixed |
People
(Reporter: kmag, Assigned: hsivonen)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
It probably doesn't do the right thing when the parent window is in a different process, but that case may be handled elsewhere.
|
Assignee | |
Updated•4 years ago
|
|
||
Comment 1•4 years ago
|
||
Tracking "Figure out GetInProcessTop usage" bugs for Fission M6b.
|
||
Updated•4 years ago
|
|
|
Assignee | |
Comment 2•4 years ago
|
||
I believe we need to obtain the parent BrowsingContext and, if it is out-of-process, request the relevant next steps to run in the process where that BrowsingContext resides.
|
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 3•4 years ago
|
||
|
|
Assignee | |
Comment 4•4 years ago
|
||
|
|
Assignee | |
Comment 5•4 years ago
|
||
|
|
Assignee | |
Comment 6•4 years ago
|
||
|
|
Assignee | |
Comment 7•4 years ago
|
||
As I iterate on this more, my new Fission code ends up doing the same thing that the e10s code was already doing--also in Fission--so maybe this whole patch is unnecessary... But let's try making the code look consistent between in-process and out-out-process iframes even if the patch ends up being pretty much a no-op really.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=5b36940eb86044465de379701bf76c21297e8465
|
|
Assignee | |
Comment 8•4 years ago
|
||
|
|
Assignee | |
Comment 9•4 years ago
|
||
If I'm reading treeherder right, the remaining oranges are known tier-2 permaoranges.
|
|
Assignee | |
Comment 10•4 years ago
|
||
After addressing review comments:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=7ba6b8003da66d83b8ec298ba01b258933bd9282
|
||
Comment 11•4 years ago
|
||
Pushed by hsivonen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/cd3fdb184761 Use BrowsingContext (and IPC) in place of GetInProcessParent usage in nsGlobalWindowOuter::FocusOuter. r=nika
|
||
Comment 12•4 years ago
|
||
| bugherder | ||
Description
•