400589 - Docshells created when reading mail are not cleaned up until shutdown

Status: RESOLVED WORKSFORME

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Peter Weilbacher]

Attachment: leak-gauge output

Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.9a9pre) Gecko/2007102003 SeaMonkey/2.0a1pre

When running with NSPR_LOG_MODULES=DOMLeak:5,DocumentLeak:5,nsDocShellLeak:5 and then running the output through leak-gauge.pl I get the attached result. It seems that all docshells opened by MailNews for the bodies of messages are only cleaned up on shutdown. At the time I ran leak-gauge I only had one browser window open (MailNews closed).

Apart from the OS/2 build quoted above this also happens on Linux with current trunk builds. It does not matter if the messages are read using the message pane or in an extra window, so this is different from bug 399027. Maybe there is a relation to bug 355005 or bug 361235.

In effect, this causes SeaMonkey memory usage for MailNews to always increase and forces the user to restart the app sooner than intended.

Comment 1

[Wayne Mery (:wsmwk)]

Peter, still see this with current 2.0b?

Comment 2

[WADA:World Anti-bad-Duping Agency]

> rv:1.9a9pre) Gecko/2007102003 SeaMonkey/2.0a1pre

Note-1:
Bug 329876 is for "virtual memory size increase(10KB) per a mail view" on MS Windows. It was WORKSFORME with Tb 2007-07-17 build. (I didn't check with Sm builds around 7/17.)
Some changes by Bug 379070 were landed in June/July 2007, and Bug 379070 was finally FIXED on 2008-11-04.

Note-2:
Bug 388353 exists for issue of "leak of resource relates to mail display", and is still open.
> Bug 388353 Displaying a Mail Messages leaks a jsContext & a global window

Comment 3

[Peter Weilbacher]

No, I checked with "Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.9.1.2pre) Gecko/20090726 SeaMonkey/2.0b2pre" that I do not see this any more. The MailNews related docshells remain for a few seconds after the last MailNews window was closed, but then they disappear from the leak-gauge output.

Great. :-) --> WFM