Last Comment Bug 735109 - unregister message listeners on shutdown in Webapps.jsm
: unregister message listeners on shutdown in Webapps.jsm
Product: Core
Classification: Components
Component: DOM (show other bugs)
: Other Branch
: All All
-- normal (vote)
: mozilla13
Assigned To: [:fabrice] Fabrice Desré
: Andrew Overholt [:overholt]
: 725533 (view as bug list)
Depends on:
  Show dependency treegraph
Reported: 2012-03-12 16:50 PDT by [:fabrice] Fabrice Desré
Modified: 2012-03-13 15:06 PDT (History)
3 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch (1.41 KB, patch)
2012-03-12 16:50 PDT, [:fabrice] Fabrice Desré
felipc: review+
Details | Diff | Splinter Review

Description User image [:fabrice] Fabrice Desré 2012-03-12 16:50:36 PDT
Created attachment 605221 [details] [diff] [review]

We leak without that.
Comment 1 User image :Felipe Gomes (needinfo me!) 2012-03-12 17:20:06 PDT
Comment on attachment 605221 [details] [diff] [review]

Very close, I tested with this patch and we also need to clear ppmm to null on shutdown, then the leak is gone!
Comment 2 User image [:fabrice] Fabrice Desré 2012-03-12 17:35:09 PDT
Comment 3 User image Marco Bonardo [::mak] 2012-03-13 05:45:32 PDT
Comment 4 User image :Felipe Gomes (needinfo me!) 2012-03-13 15:06:16 PDT
*** Bug 725533 has been marked as a duplicate of this bug. ***

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