333364 - Memory leak in DOM Inspector on 1.8 branch

Status: RESOLVED DUPLICATE of bug 336961

(Other Applications :: DOM Inspector, defect)

Description

[Pavel Cvrcek [:JasnaPaka]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20060406 Firefox/2.0a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20060406 Firefox/2.0a1

1. Run Firefox on new profile.
2. Choose Tools -> DOM Inspector im mainmenu.
3. Close DOM Inspector and Firefox


Reproducible: Always

Actual Results:  
Leaked outer window 14e6430 at address 14e6430.
Leaked inner window 14fd0a8 (outer 14e6430) at address 14fd0a8.
 ... with URI "about:blank".
Leaked document at address 27402d0.
 ... with URI "chrome://inspector/content/extensions/titledSplitter.xml".
 ... with URI "jar:file:///C:/Programy/Mozilla%20Firefox%202/extensions/[e-mail]/chrome/inspector.jar!/content/inspector/extensions/titledSplitter.xml".
Leaked document at address 27847f0.
Leaked document at address 268f4d0.
Leaked document at address 27a6c90.
Leaked document at address 261e7b0.
 ... with URI "jar:file:///C:/Programy/Mozilla%20Firefox%202/extensions/[e-mail]/chrome/inspector.jar!/content/inspector/inspector.xml".
 ... with URI "chrome://inspector/content/inspector.xml".
Leaked document at address 2758858.
 ... with URI "about:blank".
Summary:
Leaked 2 out of 15 DOM Windows
Leaked 6 out of 43 documents
Leaked 0 out of 7 docshells

Comment 1

[dolphinling]

Confirming, 1.8 branch only.

This grows each time you open the DOM Inspector (Leaked 2+4n out of .. documents where n = however many times you open it)

Comment 2

[timeless]

does this happen w/ suite?

Comment 3

[Pavel Cvrcek [:JasnaPaka]]

(In reply to comment #2)
> does this happen w/ suite?

Same problem with SeaMonkey 1.1a.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1a2) Gecko/20060518 SeaMonkey/1.1a

All works fine with SeaMonkey 1.5a.

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060521 SeaMonkey/1.5a

Comment 4

[Adam Guthrie]

Is this not a dupe of bug 336961 (which has yet to be landed on the branch). Leak Monitor shows it's leaking the observer from dom.js...

Comment 5

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

  PrefUtils.addObserver("inspector", PrefChangeObserver);
without any removeObserver call seems suspicious.  Is the PrefChangeObserver what leak-monitor shows as leaking?

Seems like a duplicate of bug 336961.

Comment 6

[Adam Guthrie]

(In reply to comment #5)
> Is the PrefChangeObserver what leak-monitor shows as leaking?

Yep. Duping.



*** This bug has been marked as a duplicate of 336961 ***

Comment 7

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Clearing flag on bug marked duplicate (duplicate is plussed too).