399027 - "mailnews.reuse_message_window = true" leaks <about:blank> DOM windows, when reading multiple mails (with message pane off)

Status: NEW

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Peter Weilbacher]

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a9pre) Gecko/2007100201 SeaMonkey/2.0a1pre

I am running my MailNews window with the Message Pane disabled (F8). So, to read messages, I double click on them to have them appear in an extra window. These extra windows can leak an about:blank document as reported by leak-gauge.pl after shutting down SeaMonkey.

To reproduce:
- get and set up leak-gauge.pl
- start SeaMonkey, open the MailNews window
- make sure the message pane is switched off
- click of the Inbox of an account (I used one of my
  IMAP accounts)
- I was asked for the master password, but I don't think
  that matters for this bug.
- double click on one message, a new window opens.
- if you shut down SM now, no leak is found
- double click on another message in the list (without
  closing the extra window)
- it is displayed in the same window
- if you shut SM down now, the leak-gauge.pl reports
  something like this:

   Leaked outer window 9508500 at address 9508500.
   Leaked inner window 95f52d8 (outer 9508500) at 
      address 95f52d8.
    ... with URI "about:blank".
   Summary:
   Leaked 2 out of 21 DOM Windows
   Leaked 0 out of 97 documents
   Leaked 0 out of 9 docshells

Comment 1

[Serge Gautherie (:sgautherie)]

(In reply to comment #0)
> 
> - double click on another message in the list (without
>   closing the extra window)

Does closing the window work around the leak ?

> - it is displayed in the same window

Can you try using
[
user_pref("mailnews.reuse_message_window", false);
]
?

Comment 2

[Peter Weilbacher]

(In reply to comment #1)
> Does closing the window work around the leak ?

Yes, it does.

> Can you try using
> [
> user_pref("mailnews.reuse_message_window", false);
> ]
> ?

That helps, too, but only if I manually close each of these windows before Ctrl-Q.

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #2)
> Yes, it does.
> That helps, too,

First issue:
the "window reuse" usecase seems not to free the previous resource(s).

NB: Could you/someone test this with Thunderbird ?

> only if I manually close each of these windows before Ctrl-Q.

Second issue:
Ctrl+Q not freeing either ?
We'll see if it still happens when the first issue is fixed...

Comment 4

[Ian Neal]

Not blocking 2.0a1 on this, renominating for 2.0 final

Comment 5

[Ian Neal]

Not blocking 2.0 on this, further information is needed to diagnose the problem. If someone produces a patch then we will look at taking a fix for 2.0. Please nominate for wanted 2.1 if you wish.

Comment 6

[Serge Gautherie (:sgautherie)]

Peter, can you still reproduce?

Comment 7

[Robert Kaiser]

I don't see any reason to give this special priority, but it's also not unwanted, so cancelling the wanted flag.