749532 - [SeaMonkey] 3 xpinstall tests leak DOMWindow(s) and DocShell(s), worse after bug 695480

Status: RESOLVED INACTIVE

(Toolkit :: Add-ons Manager, defect, P2)

Description

[Serge Gautherie (:sgautherie)]

***** SM 2.10:

(In reply to Serge Gautherie (:sgautherie) from bug 675412 comment #5)
> http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1331186151.1331190870.
> 3486.gz&fulltext=1
> Linux comm-central-trunk debug test mochitest-other on 2012/03/07 21:55:51
> {
> [toolkit/mozapps/extensions/test/xpinstall/browser_whitelist2.js]
>   1 window(s) [url =
> http://example.com/browser/toolkit/mozapps/extensions/test/xpinstall/
> installtrigger.html?%7B%22Unsigned%20XPI%22%3A%22http%3A%2F%2Fexample.
> org%2Fbrowser%2Ftoolkit%2Fmozapps%2Fextensions%2Ftest%2Fxpinstall%2Funsigned.
> xpi%22%7D]
>   1 docShell(s)
> 
> [toolkit/mozapps/extensions/test/xpinstall/browser_whitelist5.js]
>   1 window(s) [url =
> http://example.com/browser/toolkit/mozapps/extensions/test/xpinstall/
> startsoftwareupdate.html?%20http%3A%2F%2Fexample.
> com%2Fbrowser%2Ftoolkit%2Fmozapps%2Fextensions%2Ftest%2Fxpinstall%2Funsigned.
> xpi]
>   1 docShell(s)
> 
> [toolkit/mozapps/extensions/test/xpinstall/browser_whitelist6.js]
>   1 window(s) [url =
> http://example.com/browser/toolkit/mozapps/extensions/test/xpinstall/
> installchrome.html?%20http%3A%2F%2Fexample.
> com%2Fbrowser%2Ftoolkit%2Fmozapps%2Fextensions%2Ftest%2Fxpinstall%2Funsigned.
> xpi]
>   1 docShell(s)
> }


***** SM 2.12:

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1335360502.1335367242.917.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/04/25 06:28:22
mozilla-central/75c7378c87b6

Same leaks.

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1335445198.1335452248.31019.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/04/26 05:59:58
mozilla-central/cc5254f9825f

(Bug 695480 very push!)
Firefox is fixed (per bug 675412 comment 7), SeaMonkey "regressed":
same leaks + "2 window(s) [url = about:blank]" each.

Comment 1

[Serge Gautherie (:sgautherie)]

Not sure whether this bug could be related to bug 749106, but just in case...

Comment 2

[Serge Gautherie (:sgautherie)]

A clue, hopefully: (at least wrt the regression)

{
TEST-INFO | chrome://mochitests/content/browser/toolkit/mozapps/extensions/test/xpinstall/browser_navigateaway2.js | Console message: [JavaScript Error: "can't access dead object" {file: "resource:///components/amWebInstallListener.js" line: 176}]
}

Code is
http://mxr.mozilla.org/mozilla-central/source/toolkit/mozapps/extensions/amWebInstallListener.js#175
117 Installer.prototype = {
...
126   checkAllDownloaded: function() {
...
176     if (failed.length > 0) {

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to Serge Gautherie (:sgautherie) from comment #2)
> A clue, hopefully: (at least wrt the regression)

Disregard this comment: moved to Firefox bug 675412.

Ftr, SM reports no "can't access dead obj" :-)

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to Serge Gautherie (:sgautherie) from comment #1)
> Not sure whether this bug could be related to bug 749106, but just in case...

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1336332678.1336339161.9271.gz&fulltext=1
WINNT 5.2 comm-central-trunk debug test mochitest-other on 2012/05/06 12:31:18

(As +/- expected,) Bug 749106 didn't help (enough).

Comment 5

[BMO Automation]

Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.