bug 840124 - really fix the leaks this time

2013-03-26 17:02:57 -04:00 · 2013-03-26 17:02:57 -04:00 · 05d8bbf299
--- a/browser/base/content/abouthealthreport/abouthealth.js
+++ b/browser/base/content/abouthealthreport/abouthealth.js
@ -34,6 +34,10 @@ let healthReportWrapper = {
    prefs.observe("uploadEnabled", this.updatePrefState, healthReportWrapper);
  },

+  uninit: function () {
+    prefs.ignore("uploadEnabled", this.updatePrefState, healthReportWrapper);
+  },
+
  _getReportURI: function () {
    let url = Services.urlFormatter.formatURLPref("datareporting.healthreport.about.reportUrl");
    return Services.io.newURI(url, null, null);
--- a/browser/base/content/abouthealthreport/abouthealth.xhtml
+++ b/browser/base/content/abouthealthreport/abouthealth.xhtml
@ -22,7 +22,8 @@
   <script type="text/javascript;version=1.8"
     src="chrome://browser/content/abouthealthreport/abouthealth.js" />
  </head>
-  <body onload="healthReportWrapper.init();">
+  <body onload="healthReportWrapper.init();"
+        onunload="healthReportWrapper.uninit();">
    <iframe id="remote-report"/>
  </body>
 </html>