From 05d8bbf299258338e7da1f7426b53aaa90d5ead3 Mon Sep 17 00:00:00 2001 From: Mike Connor Date: Tue, 26 Mar 2013 17:02:57 -0400 Subject: [PATCH] bug 840124 - really fix the leaks this time --- browser/base/content/abouthealthreport/abouthealth.js | 4 ++++ browser/base/content/abouthealthreport/abouthealth.xhtml | 3 ++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/browser/base/content/abouthealthreport/abouthealth.js b/browser/base/content/abouthealthreport/abouthealth.js index 09ae8f5ad96d..ad0aaa5fb094 100644 --- 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); diff --git a/browser/base/content/abouthealthreport/abouthealth.xhtml b/browser/base/content/abouthealthreport/abouthealth.xhtml index 7f6f49310a45..a24780e18cb2 100644 --- a/browser/base/content/abouthealthreport/abouthealth.xhtml +++ b/browser/base/content/abouthealthreport/abouthealth.xhtml @@ -22,7 +22,8 @@