Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Commit

Permalink
Add third argument to add/removeObserver, which is required in Firefo…
Browse files Browse the repository at this point in the history
…x 54 (#2713)
  • Loading branch information
ianb authored and jaredhirsch committed Apr 19, 2017
1 parent e1dccdc commit 2f92593
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions addon/bootstrap.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,11 +29,11 @@ const appStartupPromise = new Promise((resolve, reject) => {
const prefs = Services.prefs;
const prefObserver = {
register() {
prefs.addObserver(PREF_BRANCH, this);
prefs.addObserver(PREF_BRANCH, this, false); // eslint-disable-line mozilla/no-useless-parameters
},

unregister() {
prefs.removeObserver(PREF_BRANCH, this);
prefs.removeObserver(PREF_BRANCH, this, false); // eslint-disable-line mozilla/no-useless-parameters
},

observe(aSubject, aTopic, aData) {
Expand All @@ -48,11 +48,11 @@ const prefObserver = {

const appStartupObserver = {
register() {
Services.obs.addObserver(this, "sessionstore-windows-restored");
Services.obs.addObserver(this, "sessionstore-windows-restored", false); // eslint-disable-line mozilla/no-useless-parameters
},

unregister() {
Services.obs.removeObserver(this, "sessionstore-windows-restored");
Services.obs.removeObserver(this, "sessionstore-windows-restored", false); // eslint-disable-line mozilla/no-useless-parameters
},

observe() {
Expand Down

0 comments on commit 2f92593

Please sign in to comment.