Issue 4579 - Ignore runtime.lastError caused by wrapper

polyfill.js

Index: polyfill.js
===================================================================
--- a/polyfill.js
+++ b/polyfill.js
@@ -38,16 +38,25 @@
     "tabs.sendMessage",
     "tabs.update",
     "webNavigation.getAllFrames",
     "webRequest.handlerBehaviorChanged",
     "windows.create",
     "windows.update"
   ];
 
+  // Errors that occur only when we show an interest in the response from an
+  // API call.
+  const noFulfillmentErrors = new Set([
+    "The message port closed before a response was received.",
+    // Older versions of Chrome have a typo:
+    // https://crrev.com/c33f51726eacdcc1a487b21a13611f7eab580d6d
+    "The message port closed before a reponse was received."
+  ]);

[Sebastian Noack, 2017/10/20 01:26:30]

Perhaps, we can just use a regexp to account for this typo? Also, do you happen
to know which Chrome versions had that typo? So that we can document these
versions here, and simplify the logic once we drop support for these versions.

[Manish Jethani, 2017/10/20 01:45:32]

Yes, using a regex now.

The commit was on June 30, 2017, this means it was probably fixed in Chrome 60.
I'll just link to the commit so that way when the lowest supported version is
closer to that date we can figure it out.

+
   function wrapAPI(api)
   {
     let object = browser;
     let path = api.split(".");
     let name = path.pop();
 
     for (let node of path)
     {
@@ -71,17 +80,17 @@
       if (typeof args[args.length - 1] == "undefined")
         args.pop();
 
       return new Promise((resolve, reject) =>
       {
         func.call(object, ...args, result =>
         {
           let error = browser.runtime.lastError;
-          if (error)
+          if (error && !noFulfillmentErrors.has(error.message))
             reject(error);
           else
             resolve(result);
         });
       });
     };
   }