Issue 5965 - Handle edge case when searching for options page tab

lib/options.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 20 matching lines @@
   browser.tabs.query({}, tabs =>
   {
     // We find a tab ourselves because Edge has a bug when quering tabs with
     // extension URL protocol:
     // https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8094141/
     // https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8604703/
     // Firefox won't let us query for moz-extension:// pages either, though
     // starting with Firefox 56 an extension can query for its own URLs:
     // https://bugzilla.mozilla.org/show_bug.cgi?id=1271354
     let fullOptionsUrl = browser.extension.getURL(optionsUrl);
-    callback(tabs.find(element => element.url == fullOptionsUrl));
+    let optionsTab = tabs.find(tab => tab.url == fullOptionsUrl);
+
+    if (optionsTab)
+      callback(optionsTab);
+    else
+    {
+      // It seems that the url property isn't always set for the tab with
+      // Firefox 57 on Windows 10 until the tab has finished loading. Until
+      // then the url is given as "about:blank", but the title is the URL
+      // we expect (minus the protocol part).
+      let {hostname, pathname} = new URL(fullOptionsUrl);
+      let potentialOptionsTab = tabs.find(
+        tab => tab.url == "about:blank" &&
+               tab.title == hostname + pathname &&
+               tab.status == "loading"
+      );
+
+      if (potentialOptionsTab)
+      {
+        let onRemoved;
+        let updateListener = (tabId, changeInfo, tab) =>
+        {
+          if (tabId == potentialOptionsTab.id &&
+              changeInfo.status == "complete")
+          {
+            browser.tabs.onUpdated.removeListener(updateListener);
+            browser.tabs.onRemoved.removeListener(onRemoved);
+            if (tab.url == fullOptionsUrl)
+              callback(tab);
+          }
+        };
+        browser.tabs.onUpdated.addListener(updateListener);
+        onRemoved = removedTabId =>
+        {
+          if (removedTabId == potentialOptionsTab.id)
+          {
+            browser.tabs.onUpdated.removeListener(updateListener);
+            browser.tabs.onRemoved.removeListener(onRemoved);
+            callback();
+          }
+        };
+        browser.tabs.onRemoved.addListener(onRemoved);
+      }
+      else
+        callback();
+    }
   });
 }
 
 function returnShowOptionsCall(optionsTab, callback)
 {
   if (!callback)
     return;
 
   if (optionsTab)
   {
@@ skipping 91 matching lines @@
         args: [
           {
             host: getDecodedHostname(currentPage.url).replace(/^www\./, ""),
             whitelisted: !!checkWhitelisted(currentPage)
           }
         ]
       });
     });
   });
 });