Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: lib/prefs.js

Issue 29371763: Issue 4795 - Use modern JavaScript syntax (Closed)
Left Patch Set: "use strict"; Created Jan. 16, 2017, 3:30 a.m.
Right Patch Set: Addressed some more feedback Created Jan. 18, 2017, 11:44 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « lib/popupBlocker.js ('k') | lib/requestBlocker.js » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 /* 1 /*
2 * This file is part of Adblock Plus <https://adblockplus.org/>, 2 * This file is part of Adblock Plus <https://adblockplus.org/>,
3 * Copyright (C) 2006-2016 Eyeo GmbH 3 * Copyright (C) 2006-2016 Eyeo GmbH
4 * 4 *
5 * Adblock Plus is free software: you can redistribute it and/or modify 5 * Adblock Plus is free software: you can redistribute it and/or modify
6 * it under the terms of the GNU General Public License version 3 as 6 * it under the terms of the GNU General Public License version 3 as
7 * published by the Free Software Foundation. 7 * published by the Free Software Foundation.
8 * 8 *
9 * Adblock Plus is distributed in the hope that it will be useful, 9 * Adblock Plus is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the 11 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
12 * GNU General Public License for more details. 12 * GNU General Public License for more details.
13 * 13 *
14 * You should have received a copy of the GNU General Public License 14 * You should have received a copy of the GNU General Public License
15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>. 15 * along with Adblock Plus. If not, see <http://www.gnu.org/licenses/>.
16 */ 16 */
17 17
18 /** @module prefs */ 18 /** @module prefs */
19 19
20 "use strict"; 20 "use strict";
21 21
22 let {EventEmitter} = require("events"); 22 const {EventEmitter} = require("events");
23 23
24 const keyPrefix = "pref:"; 24 const keyPrefix = "pref:";
25 25
26 let eventEmitter = new EventEmitter(); 26 let eventEmitter = new EventEmitter();
27 let overrides = Object.create(null); 27 let overrides = Object.create(null);
28 28
29 /** @lends module:prefs.Prefs */ 29 /** @lends module:prefs.Prefs */
30 let defaults = Object.create(null); 30 let defaults = Object.create(null);
31 31
32 /** 32 /**
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
183 * @static 183 * @static
184 */ 184 */
185 let Prefs = exports.Prefs = { 185 let Prefs = exports.Prefs = {
186 /** 186 /**
187 * Adds a callback that is called when the 187 * Adds a callback that is called when the
188 * value of a specified preference changed. 188 * value of a specified preference changed.
189 * 189 *
190 * @param {string} preference 190 * @param {string} preference
191 * @param {function} callback 191 * @param {function} callback
192 */ 192 */
193 on: (preference, callback) => 193 on(preference, callback)
194 { 194 {
195 eventEmitter.on(preference, callback); 195 eventEmitter.on(preference, callback);
196 }, 196 },
197 197
198 /** 198 /**
199 * Removes a callback for the specified preference. 199 * Removes a callback for the specified preference.
200 * 200 *
201 * @param {string} preference 201 * @param {string} preference
202 * @param {function} callback 202 * @param {function} callback
203 */ 203 */
204 off: (preference, callback) => 204 off(preference, callback)
205 { 205 {
206 eventEmitter.off(preference, callback); 206 eventEmitter.off(preference, callback);
207 }, 207 },
208 208
209 /** 209 /**
210 * A promise that is fullfilled when all preferences have been loaded. 210 * A promise that is fullfilled when all preferences have been loaded.
211 * Wait for this promise to be fulfilled before using preferences during 211 * Wait for this promise to be fulfilled before using preferences during
212 * extension initialization. 212 * extension initialization.
213 * 213 *
214 * @type {Promise} 214 * @type {Promise}
(...skipping 10 matching lines...) Expand all
225 } 225 }
226 226
227 function prefToKey(pref) 227 function prefToKey(pref)
228 { 228 {
229 return keyPrefix + pref; 229 return keyPrefix + pref;
230 } 230 }
231 231
232 function addPreference(pref) 232 function addPreference(pref)
233 { 233 {
234 Object.defineProperty(Prefs, pref, { 234 Object.defineProperty(Prefs, pref, {
235 get: () => (pref in overrides ? overrides : defaults)[pref], 235 get() { return (pref in overrides ? overrides : defaults)[pref]; },
236 set: value => 236 set(value)
237 { 237 {
238 let defaultValue = defaults[pref]; 238 let defaultValue = defaults[pref];
239 239
240 if (typeof value != typeof defaultValue) 240 if (typeof value != typeof defaultValue)
241 throw new Error("Attempt to change preference type"); 241 throw new Error("Attempt to change preference type");
242 242
243 if (value == defaultValue) 243 if (value == defaultValue)
244 { 244 {
245 delete overrides[pref]; 245 delete overrides[pref];
246 ext.storage.remove(prefToKey(pref)); 246 ext.storage.remove(prefToKey(pref));
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
311 eventEmitter.emit(pref); 311 eventEmitter.emit(pref);
312 } 312 }
313 } 313 }
314 }); 314 });
315 } 315 }
316 316
317 Prefs.untilLoaded = Promise.all([localLoaded, managedLoaded]).then(onLoaded); 317 Prefs.untilLoaded = Promise.all([localLoaded, managedLoaded]).then(onLoaded);
318 } 318 }
319 319
320 init(); 320 init();
LEFTRIGHT

Powered by Google App Engine
This is Rietveld