Closed
Bug 454524
Opened 16 years ago
Closed 16 years ago
New "Browser (Chrome)" (8 KB) mlk on 2008-09-09
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(Not tracked)
VERIFIED
DUPLICATE
of bug 391318
People
(Reporter: sgautherie, Unassigned)
References
Details
(Keywords: memory-leak, regression, Whiteboard: [see comment 5])
Attachments
(1 file)
(deleted),
text/plain
|
Details |
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220978283.1220984059.13971.gz
Linux comm-central dep unit test on 2008/09/09 09:38:03
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220991568.1220997331.17590.gz#err37
Linux comm-central dep unit test on 2008/09/09 13:19:28
TEST-UNEXPECTED-FAIL | runtests-leaks | leaked 8620 bytes during test execution (should have leaked no more than 0 bytes)
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220981107.1220986059.19286.gz
MacOSX 10.4 comm-central dep unit test on 2008/09/09 10:25:07
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220991568.1220996486.14976.gz#err61
MacOSX 10.4 comm-central dep unit test on 2008/09/09 13:19:28
TEST-UNEXPECTED-FAIL | runtests-leaks | leaked 8712 bytes during test execution (should have leaked no more than 0 bytes)
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220980366.1220984306.14710.gz
Win2k3 comm-central dep unit test on 2008/09/09 10:12:46
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1220992491.1220996367.14764.gz#err56
Win2k3 comm-central dep unit test on 2008/09/09 13:34:51
*** End BrowserChrome Test Results ***
TEST-UNEXPECTED-FAIL | runtests-leaks | leaked 8577 bytes during test execution (should have leaked no more than 0 bytes)
Flags: blocking-seamonkey2.0a1?
Reporter | ||
Updated•16 years ago
|
Summary: New "Browser Chrome" mlk on 2009-09-09 → New "Browser Chrome" mlk on 2008-09-09
Comment 1•16 years ago
|
||
As long as one test of that suite is failing (bug 454513), I don't care about the leaks there, because it might just as well be a leak that's caused by the test failure itself.
Flags: blocking-seamonkey2.0a1?
Reporter | ||
Comment 2•16 years ago
|
||
(In reply to comment #1)
Could be linked ... though the leak started "after" the test landed.
Depends on: 454513
Reporter | ||
Comment 3•16 years ago
|
||
Confirming: bug 454513 is fixed ... and this leak is still there.
No longer depends on: 454513
Flags: blocking-seamonkey2.0a1?
Comment 4•16 years ago
|
||
Judging from the size of the leak, this sounds very much like the known-to-happen bug 391318 issue.
It'd probably interesting to see why we didn't leak up to a certain point and then started to trigger this one.
In any case, not blocking an Alpha by a mere small leak.
Flags: blocking-seamonkey2.0a1? → blocking-seamonkey2.0a1-
Reporter | ||
Comment 5•16 years ago
|
||
Here are the c-c and m-c timeframes:
http://hg.mozilla.org/comm-central/pushloghtml?startdate=2008-09-09+05%3A56%3A49&enddate=2008-09-09+11%3A31%3A49
http://hg.mozilla.org/mozilla-central/pushloghtml?startdate=2008-09-09+10%3A22%3A49&enddate=2008-09-09+13%3A16%3A29
(In reply to comment #4)
> It'd probably interesting to see why we didn't leak up to a certain point and
> then started to trigger this one.
Exactly my point...
Reporter | ||
Updated•16 years ago
|
Flags: wanted-seamonkey2?
Summary: New "Browser Chrome" mlk on 2008-09-09 → New "Browser (Chrome)" (8 KB) mlk on 2008-09-09
Whiteboard: [see comment 5]
Comment 6•16 years ago
|
||
The wanted flag is mainly for (large) features that don't need to be fixed for a release but should be tracked as something we want to see in the final.
This is no large feature, this is "just" another possibly real bug. Given how unsure we are what bug 391318 means in reality, i.e. apart from testing machines, and that this is probably "just" another case of this, it's surely no wanted feature - it might qualify for blocking if we find the underlying issue to be actually impacting users. The latter is very unsure at the moment, so I wouldn't even request blocking for this one, but we surely should investigate the underlying problem.
Flags: wanted-seamonkey2? → wanted-seamonkey2-
Reporter | ||
Comment 7•16 years ago
|
||
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1b1pre) Gecko/20080927115844 SeaMonkey/2.0a1pre] (home, optim default) (W2Ksp4)
I built with
{
rev:16fab37cd726
m-c:4a1ad24a36bc
}
and verified that this leak is actually bug 391318.
(In reply to comment #5)
> (In reply to comment #4)
> > It'd probably interesting to see why we didn't leak up to a certain point and
> > then started to trigger this one.
(As my build is "already" leaking with these revs,)
I have no idea why the tinderboxes did not report it before they did :-(
(I don't remember if I "used" to see this leak report locally or not:
if not, it may have been because I tested with the workaround already applied :-|)
***
Unless you want to try and test on the tinderbox themselves, (or someone else can reproduce)
I suggest to resolve this bug as "fixed", now that the leak cause is confirmed (and the thresholds updated).
No longer blocks: SmTestLeak, 450983
Comment 8•16 years ago
|
||
No, leak not fixed; but is caused By 391318 (per your last comment); so duping to that.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
Reporter | ||
Comment 9•16 years ago
|
||
V.Duplicate, then.
Status: RESOLVED → VERIFIED
No longer depends on: 391318
You need to log in
before you can comment on or make changes to this bug.
Description
•