471792 - xpcshell-tests: test_privatebrowsingwrapper_removeDataFromDomain.js and test_removeDataFromDomain.js intermittently leak a few nsLocalFile + nsStringBuffer

Status: RESOLVED WORKSFORME

(Firefox :: Private Browsing, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090101 Minefield/3.2a1pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/e807ec425ad7)

I ran |make check| 3 times,
and the second time (only) I got a leak,
which I had never seen before.

Comment 1

[(no longer active)]

Could you try to reduce this by removing tests from test_removeDataFromDomain.js to see exactly which piece of code is the culprit?

Comment 2

[Serge Gautherie (:sgautherie)]

Attachment: test_removeDataFromDomain.js.log

Comment 3

[Serge Gautherie (:sgautherie)]

I tried to reproduce by running it |SOLO_FILE=| a few times, but could not.

I filed this bug in case it happens again on my local,
or after bug 569523 is fixed.

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)
> or after bug 569523 is fixed.

Bug 469523 !

Comment 5

[(no longer active)]

I tried running the tests ~20 times on a debug build with memleak detection support turned on, but didn't see this leak at all.

Comment 6

[(no longer active)]

It would be nice to see if other people can reproduce this.

Comment 7

[Mike Connor [:mconnor]]

Resolving WFM, if there's a real STR to be found, please report it then.

Comment 8

[Serge Gautherie (:sgautherie)]

V.WFM, it never happened again :-/

Comment 9

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090402 Minefield/3.6a1pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/c4f6bb1db611)

Ftr,
while testing my patch for bug 384339, I got once
{
test_privatebrowsing\unit\test_privatebrowsingwrapper_removeDataFromDomain.js

2 nsLocalFile + 2 nsStringBuffer
}
running the whole xpcshell-tests suite.
But I was enable to reproduce it running only this test.

Comment 10

[Serge Gautherie (:sgautherie)]

Ftr, a Linux build I checked on 2009.04.09 had it on its 'check' run (only?).

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1240742535.1240748437.9461.gz&fulltext=1
Linux mozilla-central unit test on 2009/04/26 03:42:15
'check' only.

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1240771335.1240778234.25540.gz&fulltext=1
Linux mozilla-central unit test on 2009/04/26 11:42:15
None.

Intermittent!

Comment 11

[Serge Gautherie (:sgautherie)]

What now? V.Fixed -> New was not possible :-(

Comment 12

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1241536379.1241540553.1631.gz&fulltext=1
Linux mozilla-1.9.1 unit test on 2009/05/05 08:12:59

Leaked 3 + 3.

Comment 13

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1246880126.1246884253.1313.gz&fulltext=1
Linux mozilla-1.9.1 unit test on 2009/07/06 04:35:26
Leaked 1 + 1.
And test_privatebrowsingwrapper_removeDataFromDomain.js leaked 3 + 3.

http://tinderbox.mozilla.org/showlog.cgi?log=Firefox3.5/1246880129.1246884688.5893.gz&fulltext=1
WINNT 5.2 mozilla-1.9.1 unit test on 2009/07/06 04:35:29
Leaked 2 + 3.

Comment 15

[(no longer active)]

This has not been reported in the Tinderboxen for a long time.  Resolving as WFM.  Please reopen if it's seen again.