Closed
Bug 1131313
Opened 11 years ago
Closed 11 years ago
outer-window-destroyed notifications aren't received in a frame script when a browser gets destroyed with e10s off
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
DUPLICATE
of bug 1126089
| Tracking | Status | |
|---|---|---|
| e10s | m5+ | --- |
People
(Reporter: MattN, Assigned: billm)
References
Details
Attachments
(1 file)
|
1.03 KB,
patch
|
Details | Diff | Splinter Review |
Listening for outer-window-destroyed from a frame script doesn't work when e10s is disabled. Notifications are received but none have the matching outerWindowID from content.QueryInterface(Ci.nsIInterfaceRequestor).getInterface(Ci.nsIDOMWindowUtils).outerWindowID;
I get a notification with a matching outerWindowID when e10s is on though.
| Reporter | ||
Comment 1•11 years ago
|
||
I we decide not to fix this then https://developer.mozilla.org/en-US/Firefox/Multiprocess_Firefox/Limitations_of_frame_scripts#Observers_in_the_content_process should be updated.
Comment 2•11 years ago
|
||
so we kill the frame script before the relevant outer-window-destroyed is fired?
| Reporter | ||
Comment 3•11 years ago
|
||
Yeas, I think that's what I saw IIRC.
Updated•11 years ago
|
Assignee: nobody → wmccloskey
Updated•11 years ago
|
Assignee: wmccloskey → gkrizsanits
| Assignee | ||
Comment 4•11 years ago
|
||
I'm actually working on this right now Gabor.
Assignee: gkrizsanits → wmccloskey
Comment 5•11 years ago
|
||
(In reply to Bill McCloskey (:billm) from comment #4)
> I'm actually working on this right now Gabor.
Whoops, sorry about that, I though this was the one you wanted to hand it over to me...
Comment 6•11 years ago
|
||
Bill, should this be fixed by bug 1126089? should we dupe it?
Flags: needinfo?(wmccloskey)
| Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: needinfo?(wmccloskey)
Resolution: --- → DUPLICATE
Updated•7 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•