Issue 6476 - Update adblockplusui dependencies to ead38c2013b5

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
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 /** @module options */
 
 "use strict";
 
 const {getDecodedHostname} = require("url");
 const {checkWhitelisted} = require("whitelisting");
-const {port} = require("messaging");
 const info = require("info");
 
 const optionsUrl = "options.html";
 
 function findOptionsTab(callback)
 {
   browser.tabs.query({}, tabs =>
   {
     // We find a tab ourselves because Edge has a bug when quering tabs with
     // extension URL protocol:
@@ skipping 65 matching lines @@
   else
   {
     // If we don't already have an options page, it means we've just opened
     // one, in which case we must find the tab, wait for it to be ready, and
     // then return the call.
     findOptionsTab(tab =>
     {
       if (!tab)
         return;
 
-      function onMessage(message, sender)
+      function onMessage(message, port)
       {
-        if (message.type == "app.listen" &&
-            sender.page && sender.page.id == tab.id)
-        {
-          port.off("app.listen", onMessage);
-          callback(new ext.Page(tab));
-        }
+        if (message.type != "app.listen")
+          return;
+
+        port.onMessage.removeListener(onMessage);
+        callback(new ext.Page(tab), port);
       }
 
-      port.on("app.listen", onMessage);
+      function onConnect(port)
+      {
+        let {name, sender} = port;

[Sebastian Noack, 2018/03/23 00:01:48]

Nit: I wouldn't care to destructure the port here, and just use port.name and
port.sender below.

[saroyanm, 2018/03/23 12:03:18]

Done.

+        if (name != "ui" || !sender || sender.tab.id != tab.id)

[Sebastian Noack, 2018/03/23 00:01:48]

Under which circumstances do we need to check for !sender?

[saroyanm, 2018/03/23 11:26:47]

I assume because this is optional, but according to the doc. this should be
presented on onConnect, I'll remove it.
Good point.

[saroyanm, 2018/03/23 12:03:17]

Done.

+          return;
+
+        browser.runtime.onConnect.removeListener(onConnect);
+        port.onMessage.addListener(onMessage);
+      }
+
+      browser.runtime.onConnect.addListener(onConnect);
     });
   }
 }
 
 let showOptions =
 /**
  * Opens the options page.
  *
  * @param {function} callback
  */
@@ skipping 59 matching lines @@
 }
 
 // On Firefox for Android, open the options page directly when the browser
 // action is clicked.
 browser.browserAction.onClicked.addListener(() =>
 {
   browser.tabs.query({active: true, lastFocusedWindow: true}, ([tab]) =>
   {
     let currentPage = new ext.Page(tab);
 
-    showOptions(optionsPage =>
+    showOptions((optionsPage, port) =>
     {
       if (!/^https?:$/.test(currentPage.url.protocol))
         return;
 
-      optionsPage.sendMessage({
+        port.postMessage({

[Sebastian Noack, 2018/03/23 00:01:48]

Nit: The indentation is off here.

[saroyanm, 2018/03/23 12:03:18]

Done.

         type: "app.respond",
         action: "showPageOptions",
         args: [
           {
             host: getDecodedHostname(currentPage.url).replace(/^www\./, ""),
             whitelisted: !!checkWhitelisted(currentPage)
           }
         ]
       });
     });
   });
 });