Unified Diff: chrome/content/ui/filters-subscriptionactions.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.
« no previous file with comments
|
« no previous file
|
chrome/content/ui/progressBar.js » ('j')
|
lib/filterStorage.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: chrome/content/ui/filters-subscriptionactions.js |
| =================================================================== |
| --- a/chrome/content/ui/filters-subscriptionactions.js |
| +++ b/chrome/content/ui/filters-subscriptionactions.js |
| @@ -231,17 +231,17 @@ var SubscriptionActions = |
| let result = this._ctrlMask; |
| try { |
| let accelKey = Utils.prefService.getIntPref("ui.key.accelKey"); |
| if (accelKey == Ci.nsIDOMKeyEvent.DOM_VK_META) |
| result = this._metaMask; |
| else if (accelKey == Ci.nsIDOMKeyEvent.DOM_VK_ALT) |
| result = this._altMask; |
| } catch(e) {} |
| - this.__defineGetter__("_accelMask", function() result); |
| + Object.defineProperty(this, "_accelMask", {value: result}); |
|
Wladimir Palant
2014/06/23 11:12:41
I wonder whether not having these properties enume
|
| return result; |
| }, |
| /** |
| * Called when a key is pressed on the subscription list. |
| */ |
| keyPress: function(/**Event*/ event) |
| { |
« no previous file with comments
|
« no previous file
|
chrome/content/ui/progressBar.js » ('j')
|
lib/filterStorage.js » ('J')
