Unified Diff: lib/content/snippets.js
Issue 30033574:
Issue 7419 - Allow wrapping function for abort-on-property-* (Closed)
Base URL: https://hg.adblockplus.org/adblockpluscore/
Patch Set: Adde more tests
Created March 29, 2019, 2:34 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
test/browser/snippets.js » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: lib/content/snippets.js |
| =================================================================== |
| --- a/lib/content/snippets.js |
| +++ b/lib/content/snippets.js |
| @@ -711,17 +711,17 @@ |
| descriptor.configurable = true; |
| Object.defineProperty(object, property, descriptor); |
| return; |
| } |
| let name = property.slice(0, dotIndex); |
| property = property.slice(dotIndex + 1); |
| let value = object[name]; |
| - if (value && typeof value == "object") |
| + if (value && (typeof value == "object" || typeof value == "function")) |
| wrapPropertyAccess(value, property, descriptor); |
| let currentDescriptor = Object.getOwnPropertyDescriptor(object, name); |
| if (currentDescriptor && !currentDescriptor.configurable) |
| return; |
| let setter = newValue => |
| { |
« no previous file with comments
|
« no previous file
|
test/browser/snippets.js » ('j')
|
no next file with comments »
