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

RESOLVED FIXED in mozilla15

Status

defect
--
major
RESOLVED FIXED
10 years ago
9 months ago

People

(Reporter: sgautherie, Assigned: sgautherie)

Tracking

(Blocks 1 bug, {memory-leak})

Trunk
mozilla15
x86
Windows 2000
Dependency tree / graph

SeaMonkey Tracking Flags

(seamonkey2.9 wontfix, seamonkey2.10 fixed, seamonkey2.11 fixed, seamonkey2.12 fixed)

Details

Attachments

(2 attachments)

[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...
Posted file Leak Log
Blocks: SmTestLeak
No longer blocks: 504480
Depends on: 504480
[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
}
Depends on: 508280
No longer depends on: 508280
Depends on: 533358
[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.
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #619606 - Flags: review?(iann_bugzilla)
Attachment #619606 - Flags: approval-comm-beta?
Attachment #619606 - Flags: approval-comm-aurora?
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.
Attachment #619606 - Flags: review?(bugspam.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?
Attachment #619606 - Flags: review?(bugspam.Callek) → review+
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.
Attachment #619606 - Attachment description: (Av1) test_idcheck.xul: Re-enable Venkman test → (Av1) test_idcheck.xul: Re-enable Venkman test [Checked in: Comment 6]
Attachment #619606 - Flags: review?(iann_bugzilla)
Hum, test succeeds, but can't confirm no leak until bug 603110 is fixed :-/
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
Attachment #619606 - Flags: approval-comm-beta?
Attachment #619606 - Flags: approval-comm-beta+
Attachment #619606 - Flags: approval-comm-aurora?
Attachment #619606 - Flags: approval-comm-aurora+
Keywords: checkin-needed
Whiteboard: [c-n: ca2d953c5271 to c-a and c-b]
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
Attachment #619606 - Attachment description: (Av1) test_idcheck.xul: Re-enable Venkman test [Checked in: Comment 6] → (Av1) test_idcheck.xul: Re-enable Venkman test [Checked in: Comments 6 and 8]
Keywords: checkin-needed
Whiteboard: [c-n: ca2d953c5271 to c-a and c-b]
Product: Other Applications → Other Applications Graveyard
You need to log in before you can comment on or make changes to this bug.