After our AppCore has been XPConnected, we need to merge MsgAppCore and MsgMailSession (since they are both singleton interfaces into messenger) Then, each Messenger-oriented window should include a messenger javascript file that says something like: var messenger = Components["components://netscape/messenger/session"].getService(); Then when a window closes and it's JS context goes away, the "messenger" variable will be dereferenced. When the final window goes away, it should shut down the messenger service, which should shut down the account manager, and so forth. Basically this is the universal "Messenger exits" code....this is a good place to see if we're leaking too.

Time to stop leaks. upping priority

I think putterman just checked in a bunch of stuff related to this...scott?

Yeah, I did. Marking fixed.

Par, can you or Suresh verify this when you get a chance? Thanks. Also, I think the platform should be All and not Solaris, so I'm changing this.

The fix is in /mozilla/mailnews/base/src/nsMessengerBootstrap.cpp::Shutdown() nsresult nsMessengerBootstrap::Shutdown() { nsresult finalrv = NS_OK; nsresult rv; rv = nsService::UnregisterService("component://netscape/appshell/component/messenger" ); if(NS_FAILED(rv)) finalrv = rv; rv = nsServiceManager::UnregisterService("component://netscape/messenger/services/ses sion"); if(NS_FAILED(rv)) finalrv = rv; rv = nsServiceManager::UnregisterService("component://netscape/messenger/biffManager" ); if(NS_FAILED(rv)) finalrv = rv; return finalrv; }