Unified Diff: polyfill.js
Issue 29589646:
shouldWrapAPIs in polifills breaks ABP for Edge (Closed)
Patch Set:
Created Oct. 26, 2017, 4:30 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: polyfill.js |
| =================================================================== |
| --- a/polyfill.js |
| +++ b/polyfill.js |
| @@ -117,7 +117,7 @@ |
| { |
| try |
| { |
| - return !(browser.storage.local.get([]) instanceof Promise); |
| + return !(browser.storage.local.get([], () => {}) instanceof Promise); |
|
Sebastian Noack
2017/10/26 17:11:24
This will defeat the purpose of this check, since
Oleksandr
2017/10/27 13:15:29
I am not sure what happened but as I test again no
|
| } |
| catch (error) |
| { |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
