corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1736141 - Remove unneeded observer in devtools theme code. r=jdescottes

Browse Source 
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
This commit is contained in:
Emilio Cobos Álvarez
2021-10-19 10:48:53 +00:00
parent 71c3df0c25
commit 1867997094
1 changed files with 0 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
devtools/client/shared/theme.js
View File

@@ -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 */