Issue 2381 - Added share overlay to options page

common.js

+/*
+ * This file is part of Adblock Plus <https://adblockplus.org/>,
+ * Copyright (C) 2006-2015 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/>.
+ */
+

[saroyanm, 2015/09/08 17:33:49]

I think we should use "strict" here as well, but not sure why it's missing in
background.js and i18n.js ? Is there a reason, or we forget to add ? 

[Thomas Greiner, 2015/09/23 14:03:05]

The reason for that is that adding strict mode after the fact can cause
unexpected errors. Since most of this code comes from a file that didn't use
strict mode before, I didn't add it.

Anyway, I tested all functions in this file now and added the statement.

+(function(global)
+{
+  global.E = function E(id)
+  {
+    return document.getElementById(id);
+  }
+
+  global.getDocLink = function(link, callback)
+  {
+    ext.backgroundPage.sendMessage({
+      type: "app.get",
+      what: "doclink",
+      link: link
+    }, callback);
+  }
+
+  global.checkShareResource = function(url, callback)
+  {
+    ext.backgroundPage.sendMessage(
+    {
+      type: "filters.blocked",
+      url: url,
+      requestType: "SCRIPT",
+      docDomain: "adblockplus.org",
+      thirdParty: true
+    }, callback);
+  }
+
+  global.openSharePopup = function(url)
+  {
+    var glassPane = E("glass-pane");
+    if (!glassPane)
+    {
+      glassPane = document.createElement("div");
+      glassPane.setAttribute("id", "glass-pane");
+      document.body.appendChild(glassPane);
+    }
+
+    var iframe = E("share-popup");
+    if (!iframe)
+    {
+      iframe = document.createElement("iframe");
+      iframe.setAttribute("id", "share-popup");
+      iframe.setAttribute("scrolling", "no");
+      glassPane.appendChild(iframe);
+    }
+
+    // Firefox 38+ no longer allows messaging using postMessage so we need
+    // to have a fake top level frame to avoid problems with scripts that try to
+    // communicate with the first-run page
+    var isGecko = ("Components" in window);
+    if (isGecko)
+    {
+      try
+      {
+        var Ci = Components.interfaces;
+        iframe.contentWindow
+          .QueryInterface(Ci.nsIInterfaceRequestor)
+          .getInterface(Ci.nsIDocShell)
+          .setIsBrowserInsideApp(Ci.nsIScriptSecurityManager.UNKNOWN_APP_ID);
+      }
+      catch(ex)
+      {
+        console.error(ex);
+      }
+    }
+
+    var popupMessageReceived = false;
+    function resizePopup(width, height)
+    {
+      iframe.width = width;
+      iframe.height = height;
+      iframe.style.marginTop = -height / 2 + "px";
+      iframe.style.marginLeft = -width / 2 + "px";
+      popupMessageReceived = true;
+      window.removeEventListener("message", popupMessageListener);
+    }
+
+    var popupMessageListener = function(event)
+    {
+      if (!/[.\/]adblockplus\.org$/.test(event.origin)
+        || !("width" in event.data)
+        || !("height" in event.data))
+        return;
+
+      resizePopup(event.data.width, event.data.height);
+    };
+    // Firefox requires last parameter to be true to be triggered by
+    // unprivileged pages
+    window.addEventListener("message", popupMessageListener, false, true);
+
+    var popupLoadListener = function()
+    {
+      if (!popupMessageReceived && isGecko)
+      {
+        var rootElement = iframe.contentDocument.documentElement;
+        var width = rootElement.dataset.width;
+        var height = rootElement.dataset.height;
+        if (width && height)
+          resizePopup(width, height);
+      }
+
+      if (popupMessageReceived)
+      {
+        iframe.className = "visible";

[saroyanm, 2015/09/08 17:33:49]

This implementation is inconsistent, with other similar implementation on the
page, but it's not the part of the review and it just about reuse the overlay,
so we can fix this also afterward.

[Thomas Greiner, 2015/09/23 14:03:05]

Agreed, let's work on consistency changes separately.

+
+        var popupCloseListener = function()
+        {
+          iframe.className = glassPane.className = "";
+          document.removeEventListener("click", popupCloseListener);
+        };
+        document.addEventListener("click", popupCloseListener, false);
+      }
+      else
+      {
+        glassPane.className = "";
+        window.removeEventListener("message", popupMessageListener);
+      }
+
+      iframe.removeEventListener("load", popupLoadListener);
+    };
+    iframe.addEventListener("load", popupLoadListener, false);
+
+    iframe.src = url;
+    glassPane.className = "visible";
+  }
+})(this);