471796 - nsGlobalWindows shouldn't retain objects past xpcom shutdown

Status: NEW

(Core :: DOM: Core & HTML, defect, P5)

Description

[timeless]

Attachment: release window contents at shutdown before cycle collecting

Currently this isn't really something nsGlobalWindow cares about, but it does happen. It's not much of an issue because we then try to call getservice during xpcom shutdown, which vetos it. w/ my changes, we have the observerservice and then *it* complains.

gklayout!nsGlobalWindow::~nsGlobalWindow
gklayout!nsGlobalChromeWindow::~nsGlobalChromeWindow
gklayout!nsGlobalChromeWindow::`scalar deleting destructor'
gklayout!nsGlobalWindow::Release
gklayout!nsGlobalChromeWindow::Release
xpcom_core!nsXPCOMCycleCollectionParticipant::Unroot
xpcom_core!nsCycleCollector::CollectWhite
xpcom_core!nsCycleCollector::FinishCollection
xpcom_core!nsCycleCollector_finishCollection
xpc3250!XPCCycleCollectGCCallback
js3250!js_GC
js3250!JS_GC
xpc3250!nsXPConnect::Collect
xpcom_core!nsCycleCollector::Collect
xpcom_core!nsCycleCollector::Shutdown
xpcom_core!nsCycleCollector_shutdown
xpcom_core!NS_ShutdownXPCOM_P
xul!ScopedXPCOMStartup::~ScopedXPCOMStartup
xul!XRE_main

with these changes, the objects are essentially empty shells.

i'm testing w/ thunderbird(against mozilla-central) just running thunderbird -no-remote -P and canceling the profile manager.

Comment 1

[Johnny Stenback (:jst)]

Comment on attachment 355049 [details] [diff] [review]
release window contents at shutdown before cycle collecting

Clearing out old reviews. If this is still relevant, please re-request review for this patch.

Comment 2

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1472046

Move all DOM bugs that haven’t been updated in more than 3 years and has no one currently assigned to P5.

If you have questions, please contact :mdaly.