470235 - (TUnit) xpcshell-tests test_handlerService.js and test_punycodeURIs.js leak

Status: VERIFIED FIXED

(Core Graveyard :: File Handling, defect)

Description

[Serge Gautherie (:sgautherie)]

_tests\xpcshell-simple\test_uriloader_exthandler\unit
test_handlerService.js
test_punycodeURIs.js

See bug 456414.

Comment 1

[Serge Gautherie (:sgautherie)]

Confirmed by
{
http://tinderbox.mozilla.org/showlog.cgi?log=Thunderbird/1238183877.1238186339.30647.gz&fulltext=1
Linux comm-central check on 2009/03/27 12:57:57
}

Comment 2

[Serge Gautherie (:sgautherie)]

Confirmed by
{
http://tinderbox.mozilla.org/showlog.cgi?log=Thunderbird3.0/1238257155.1238260151.7894.gz&fulltext=1
MacOSX 10.4 comm-1.9.1 check on 2009/03/28 09:19:15
}

Comment 3

[Serge Gautherie (:sgautherie)]

Ftr, test_handlerService.js leak was greatly reduced yesterday: see bug 470241 comment 3.

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #3)

Previous leak was 11-15 kB, new is 2-4 kB.

NB: This did not regress with bug 470241 comment 4.

Comment 5

[Serge Gautherie (:sgautherie)]

Ftr,
Bug 489078 comment 1:
{
[...] test_punycodeURIs.js [...] caused by the nsEnvironment::Set
calls.

http://mxr.mozilla.org/mozilla-central/source/xpcom/threads/nsEnvironment.cpp#163
}

Comment 6

[Serge Gautherie (:sgautherie)]

Bug 489078 fixed (as much as possible) the remaining leak(s):
FF leak decreased to '3 nsStringBuffer' only, as expected.
TB/Linux too, TB/MacOSX to None(!), TB/Windows is red atm(...).

V.Fixed + fixed1.9.1