504730 - [SeaMonkey] mochitest-browser-chrome, test_idcheck.xul: venkman.xul leaks 375 kB

Status: RESOLVED FIXED

(Other Applications Graveyard :: Venkman JS Debugger, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090715 SeaMonkey/2.1a1pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/d190d9b6ccd1
 +http://hg.mozilla.org/comm-central/rev/773809454cf2 + bug 503724 patch)

Venkman could (soon) pass test_idcheck.xul,
but it leaks:

{
vnk: Application venkman, 'JavaScript Debugger' loaded.
vnk: init {
vnk:   initDebugger {} initDebugger 0.411 sec
vnk: } 0.85 sec
vnk: Application venkman, 'JavaScript Debugger' unloading.

Leaked URLs:
  file:///.../bin/chrome/toolkit.jar
  file:///.../extensions/%7Bf13b157f-b174-47e7-a34d-4815ddfdfeb8%7D/chrome/venkman.jar
  [...]
  chrome://global/content/bindings/browser.xml
  x:///content/global/bindings/browser.xml
  chrome://global/content/bindings/browser.xml
  chrome://global/content/bindings/browser.xml#browser
  [and]
  chrome://venkman/content/venkman-bindings.xml
  x:///content/venkman/venkman-bindings.xml
  chrome://venkman/content/venkman-bindings.xml
  chrome://venkman/content/venkman-bindings.xml#floatingview
  chrome://venkman/content/venkman-bindings.xml#viewcontainer-box
  chrome://venkman/content/venkman-bindings.xml#viewcontainer-tab
  chrome://venkman/content/venkman-bindings.xml#little-tab
}

Hopefully, this may help (bug 198851 or) bug 350326...

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: Leak Log

Comment 2

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1.7pre) Gecko/20091207 SeaMonkey/2.0.2pre] (comm-1.9.1-win32-unittest/1260219994) (W2Ksp4)
{
TEST-UNEXPECTED-FAIL | automationutils.processLeakLog() | leaked 335955 bytes during test execution
}

Comment 3

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) test_idcheck.xul: Re-enable Venkman test [Checked in: Comments 6 and 8]

[Mozilla/5.0 (Windows NT 5.0; rv:12.0a1) Gecko/20120126 Firefox/12.0a1 SeaMonkey/2.9a1] (custom debug build for Windows 2000)

Doesn't seem to leak anymore.


[Approval Request Comment]
No risk, test only.

Comment 4

[Serge Gautherie (:sgautherie)]

Comment on attachment 619606 [details] [diff] [review]
(Av1) test_idcheck.xul: Re-enable Venkman test
[Checked in: Comments 6 and 8]

Whoever reviews this first.

Comment 5

[Justin Wood (:Callek)]

Comment on attachment 619606 [details] [diff] [review]
(Av1) test_idcheck.xul: Re-enable Venkman test
[Checked in: Comments 6 and 8]

Review of attachment 619606 [details] [diff] [review]:
-----------------------------------------------------------------

Is the leak fixed or is that what you hope to find out?

Comment 6

[Serge Gautherie (:sgautherie)]

Comment on attachment 619606 [details] [diff] [review]
(Av1) test_idcheck.xul: Re-enable Venkman test
[Checked in: Comments 6 and 8]

http://hg.mozilla.org/comm-central/rev/ca2d953c5271


(In reply to Serge Gautherie (:sgautherie) from comment #3)
> Doesn't seem to leak anymore.

(In reply to Justin Wood (:Callek) from comment #5)
> Is the leak fixed or is that what you hope to find out?

I hope tinderboxes will confirm my local run.

Comment 7

[Serge Gautherie (:sgautherie)]

Hum, test succeeds, but can't confirm no leak until bug 603110 is fixed :-/

Comment 8

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 619606 [details] [diff] [review]
(Av1) test_idcheck.xul: Re-enable Venkman test
[Checked in: Comments 6 and 8]

http://hg.mozilla.org/releases/comm-aurora/rev/f849cc3abba4
http://hg.mozilla.org/releases/comm-beta/rev/da6af1c55a32