shouldWrapAPIs in polifills breaks ABP for Edge

polyfill.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 99 matching lines @@
           }
         });
       });
     };
   }
 
   function shouldWrapAPIs()
   {
     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 on Firefox no promise is
returned if a callback is provided.

What is the behavior/issue on Microsoft Edge?

[Oleksandr, 2017/10/27 13:15:29]

I am not sure what happened but as I test again now there is just an exception
which gets caught by the catch as expected. No need for this patch then, so I am
closing this. Sorry for the trouble!

     }
     catch (error)
     {
     }
 
     return true;
   }
 
   if (shouldWrapAPIs())
   {
     // Unlike Firefox and Microsoft Edge, Chrome doesn't have a "browser" object,
     // but provides the extension API through the "chrome" namespace
     // (non-standard).
     if (typeof browser == "undefined")
       window.browser = chrome;
 
     for (let api of asyncAPIs)
       wrapAPI(api);
   }
 
   // Workaround since HTMLCollection, NodeList, StyleSheetList, and CSSRuleList
   // didn't have iterator support before Chrome 51.
   // https://bugs.chromium.org/p/chromium/issues/detail?id=401699
   for (let object of [HTMLCollection, NodeList, StyleSheetList, CSSRuleList])
   {
     if (!(Symbol.iterator in object.prototype))
       object.prototype[Symbol.iterator] = Array.prototype[Symbol.iterator];
   }
 }