Unified Diff: options.js
Issue 29371945:
Issue 4802 - Add back compatibility for Microsoft Edge after #4722 removed it (Closed)
Patch Set:
Created Jan. 16, 2017, 6:33 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« dependencies ('K')
|
« options.html ('k')
|
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: options.js |
| =================================================================== |
| --- a/options.js |
| +++ b/options.js |
| @@ -56,7 +56,12 @@ |
| message[paramKeys[i]] = arguments[i]; |
| } |
| - ext.backgroundPage.sendMessage(message, callback); |
| + // Edge silently fails when sendMessage is called with a callback |
| + // parameter of undefined, so we work around that here. |
| + if (callback) |
| + ext.backgroundPage.sendMessage(message, callback); |
| + else |
| + ext.backgroundPage.sendMessage(message); |
| }; |
| } |
