From 1867997094152a95d56354560f21c4edf8d8a5a5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= Date: Tue, 19 Oct 2021 10:48:53 +0000 Subject: [PATCH] Bug 1736141 - Remove unneeded observer in devtools theme code. r=jdescottes The toolbar pref change triggers eventually a look-and-feel-changed notification, so no need to observe that directly. Depends on D128674 Differential Revision: https://phabricator.services.mozilla.com/D128675 --- devtools/client/shared/theme.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/devtools/client/shared/theme.js b/devtools/client/shared/theme.js index dfb5755e2c0a..c9e9be4c1081 100644 --- a/devtools/client/shared/theme.js +++ b/devtools/client/shared/theme.js @@ -91,7 +91,6 @@ const setTheme = (exports.setTheme = newTheme => { */ const addThemeObserver = (exports.addThemeObserver = observer => { Services.obs.addObserver(observer, "look-and-feel-changed"); - Services.prefs.addObserver("browser.theme.toolbar-theme", observer); Services.prefs.addObserver(THEME_PREF, observer); }); @@ -100,7 +99,6 @@ const addThemeObserver = (exports.addThemeObserver = observer => { */ const removeThemeObserver = (exports.removeThemeObserver = observer => { Services.obs.removeObserver(observer, "look-and-feel-changed"); - Services.prefs.removeObserver("browser.theme.toolbar-theme", observer); Services.prefs.removeObserver(THEME_PREF, observer); }); /* eslint-enable */