Closed
Bug 1037625
Opened 10 years ago
Closed 10 years ago
TabChild destruction doesn't let consumers respond to "inner-window-destroyed" notifications before potentially killing the process
Categories
(Core :: DOM: Content Processes, defect)
Core
DOM: Content Processes
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: bent.mozilla, Assigned: bent.mozilla)
Details
Attachments
(1 file)
2.61 KB,
patch
|
khuey
:
review+
|
Details | Diff | Splinter Review |
Datastore relies on the "inner-window-destroyed" notification to unregister some message listeners in the parent process. These notifications are generated via an async event, the WindowDestroyedEvent that gets generated in nsGlobalWindow::FreeInnerObjects/NotifyWindowIDDestroyed. This event does not have a chance to run if the TabChild that is being destroyed is the final TabChild for a process. One additional bounce through the event loop of the child is enough to ensure that all async messages like this get delivered.
Attachment #8454650 -
Flags: review?(khuey)
Attachment #8454650 -
Flags: review?(khuey) → review+
Assignee | ||
Comment 1•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/2cae53b8f3f4
https://hg.mozilla.org/mozilla-central/rev/2cae53b8f3f4
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•