| 1 |
|
| 2 | # HG changeset patch
|
| 3 | # User Magnus Melin <[email protected]>
|
| 4 | # Date 1757493192 0
|
| 5 | # Node ID 1cc168c9d0a5c55744d2886aa380f5a7bf712ef4
|
| 6 | # Parent 20980bc07105ebb761347e9c99937e572eedf03a
|
| 7 | Bug 1987834 - Don't remove not added observer of AUTO_UPDATE_CHANGED_TOPIC when MOZ_UPDATER is off. r=john.bieling
|
| 8 |
|
| 9 | Differential Revision: https://phabricator.services.mozilla.com/D264376
|
| 10 |
|
| 11 | diff --git a/mail/components/preferences/general.js b/mail/components/preferences/general.js
|
| 12 | |
| 13 | |
| 14 | @@ -2175,17 +2175,19 @@ var gGeneralPane = {
|
| 15 | }
|
| 16 |
|
| 17 | return /^https?/.test(uri.scheme) ? uri.resolve("/favicon.ico") : "";
|
| 18 | },
|
| 19 |
|
| 20 | destroy() {
|
| 21 | window.removeEventListener("unload", this);
|
| 22 |
|
| 23 | - Services.obs.removeObserver(this, AUTO_UPDATE_CHANGED_TOPIC);
|
| 24 | + if (AppConstants.MOZ_UPDATER) {
|
| 25 | + Services.obs.removeObserver(this, AUTO_UPDATE_CHANGED_TOPIC);
|
| 26 | + }
|
| 27 | Services.prefs.removeObserver("mailnews.tags.", this);
|
| 28 | },
|
| 29 |
|
| 30 | // nsISupports
|
| 31 |
|
| 32 | QueryInterface: ChromeUtils.generateQI(["nsIObserver"]),
|
| 33 |
|
| 34 | // nsIObserver
|
| 35 |
|