Bug 962212 (part 1) - Add missing removeObserver calls [r=ally]
authorMatt Brubeck <mbrubeck@mozilla.com>
Mon, 27 Jan 2014 15:42:50 -0800
changeset 181488 0d1c2194dd03f7d6c1d8f4f88f69e6edc00e1eb2
parent 181487 a5d86262abb339fd3ecece74c11d739ed41181f5
child 181489 05075ab03a2b46da0fd1b3e17a41226e6acfaa9e
push id3343
push userffxbld
push dateMon, 17 Mar 2014 21:55:32 +0000
treeherdermozilla-beta@2f7d3415f79f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersally
bugs962212
milestone29.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 962212 (part 1) - Add missing removeObserver calls [r=ally]
browser/metro/base/content/browser-ui.js
--- a/browser/metro/base/content/browser-ui.js
+++ b/browser/metro/base/content/browser-ui.js
@@ -183,16 +183,21 @@ var BrowserUI = {
     messageManager.removeMessageListener("DOMWillOpenModalDialog", this);
     messageManager.removeMessageListener("DOMWindowClose", this);
 
     messageManager.removeMessageListener("Browser:OpenURI", this);
     messageManager.removeMessageListener("Browser:SaveAs:Return", this);
     messageManager.removeMessageListener("Content:StateChange", this);
 
     messageManager.removeMessageListener("Browser:MozApplicationManifest", OfflineApps);
+
+    Services.prefs.removeObserver(debugServerStateChanged, this);
+    Services.prefs.removeObserver(debugServerPortChanged, this);
+    Services.prefs.removeObserver("app.crashreporter.autosubmit", this);
+
     Services.obs.removeObserver(this, "handle-xul-text-link");
 
     PanelUI.uninit();
     FlyoutPanelsUI.uninit();
     MetroDownloadsView.uninit();
     SettingsCharm.uninit();
     PageThumbs.uninit();
     if (WindowsPrefSync) {