Last Comment Bug 742190 - Leak with MozMutationObserver
: Leak with MozMutationObserver
Status: RESOLVED FIXED
: mlk, testcase
Product: Core
Classification: Components
Component: DOM (show other bugs)
: 14 Branch
: x86_64 Mac OS X
: -- normal (vote)
: mozilla14
Assigned To: Olli Pettay [:smaug]
:
Mentors:
Depends on:
Blocks: stirdom 641821
  Show dependency treegraph
 
Reported: 2012-04-03 21:50 PDT by Jesse Ruderman
Modified: 2012-04-04 00:14 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
testcase (makes Firefox leak all the things) (352 bytes, text/html)
2012-04-03 21:50 PDT, Jesse Ruderman
no flags Details
patch (1.85 KB, patch)
2012-04-03 22:38 PDT, Olli Pettay [:smaug]
jonas: review+
Details | Diff | Review

Description Jesse Ruderman 2012-04-03 21:50:08 PDT
Created attachment 612103 [details]
testcase (makes Firefox leak all the things)

1. Run a debug build of Firefox with XPCOM_MEM_LEAK_LOG=2
2. Load the testcase
3. Quit Firefox
Result: trace-refcnt complains of leaks
Comment 1 Olli Pettay [:smaug] 2012-04-03 22:38:10 PDT
Created attachment 612106 [details] [diff] [review]
patch

Same is done already in ::ContentRemoved
Comment 2 Olli Pettay [:smaug] 2012-04-04 00:14:57 PDT
https://hg.mozilla.org/mozilla-central/rev/b59fef232efe

Note You need to log in before you can comment on or make changes to this bug.