450812 - nsExceptionManager objects destroyed long before threads are

Status: NEW

(Core :: XPCOM, defect)

Description

[Shawn Wilsher :sdwilsh]

nsExcpetionService releases all nsExceptionManagers during xpcom-shutdown.  The problem is that threads are not shutdown until xpcom-shutdown-threads, so the nsExceptionMnager object is released before the thread would normally free it.

There are two possible solutions here:
1) Add a new topic that runs after xpcom-shutdown-threads called something like xpcom-all-threads-shutdown and the nsExceptionService listens to.
2) Move nsExceptionService's shutodwn code to NS_ShutdownXPCOM like the timer service and thread manager.

I'm going to implement (1) for the time being to unblock myself.  I'll need some help if we were to do (2).

Comment 1

[Shawn Wilsher :sdwilsh]

Attachment: v0.1

This doesn't actually pass make check.  There are some weird errors that I don't really understand :(

Comment 4

[Brian Crowder]

Duping some other bugs here, since this at least has a swing at a patch....  We should figure this out, though, since it prevents bug 570619 from doing the Right Thing.

Comment 5

[Josh Matthews [:jdm]]

I threw this at a try run, and it was all green except for known oranges.  Shawn, do you recall at all what the errors you saw were?  I think this is blocking the httpd.js gc() work, so I'd like to clear this bug up.

Comment 6

[Shawn Wilsher :sdwilsh]

Uh, no idea to be honest.  Turns out threads + JS was a bad idea and I didn't need this patch because we ended up not going with that approach.