Unified Diff: lib/filterStorage.js
Issue 5055554716172288:
Issue 653 -Object.defineProperty instead of defineGetter / defineSetter (Closed)
Patch Set:
Created June 9, 2014, 9:29 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« chrome/content/ui/filters-subscriptionactions.js ('K')
|
« lib/filterClasses.js ('k')
|
lib/io.js » ('j')
|
lib/io.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/filterStorage.js |
| =================================================================== |
| --- a/lib/filterStorage.js |
| +++ b/lib/filterStorage.js |
| @@ -80,18 +80,18 @@ let FilterStorage = exports.FilterStorag |
| if (file) |
| file.append("patterns.ini"); |
| } catch(e) {} |
| } |
| if (!file) |
| Cu.reportError("Adblock Plus: Failed to resolve filter file location from extensions.adblockplus.patternsfile preference"); |
| - this.__defineGetter__("sourceFile", () => file); |
| - return this.sourceFile; |
| + Object.defineProperty(FilterStorage, "sourceFile", {value: file}); |
|
Wladimir Palant
2014/06/23 11:12:41
Keep using this rather than FilterStorage here?
|
| + return file; |
| }, |
| /** |
| * Will be set to true if no patterns.ini file exists. |
| * @type Boolean |
| */ |
| firstRun: false, |
« chrome/content/ui/filters-subscriptionactions.js ('K')
|
« lib/filterClasses.js ('k')
|
lib/io.js » ('j')
|
lib/io.js » ('J')
