Issue 5593 - Use messaging in popup for prefs, whitelisting, and stats

stats.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/>.
  */
 
 /* global i18n */
 
 "use strict";
 
 (function()
 {
   const {require} = ext.backgroundPage.getWindow();
 
   const {getBlockedPerPage} = require("stats");
   const {FilterNotifier} = require("filterNotifier");
-  const {Prefs} = require("prefs");
 
   let currentPage;
   const shareURL = "https://adblockplus.org/";
 
   let messageMark = {};
   let shareLinks = {
     facebook: ["https://www.facebook.com/dialog/feed", {
       app_id: "475542399197328",
       link: shareURL,
       redirect_uri: "https://www.facebook.com/",
@@ skipping 32 matching lines @@
       );
     }
     return url + "?" + querystring.join("&");
   }
 
   function onLoad()
   {
     document.getElementById("share-box").addEventListener("click", share,
                                                           false);
     let showIconNumber = document.getElementById("show-iconnumber");
-    showIconNumber.setAttribute("aria-checked", Prefs.show_statsinicon);
+    ext.prefs.get("show_statsinicon", showStatsInIcon =>
+    {
+      showIconNumber.setAttribute("aria-checked", showStatsInIcon);
+    });
     showIconNumber.addEventListener("click", toggleIconNumber, false);
     document.querySelector("label[for='show-iconnumber']").addEventListener(
       "click", toggleIconNumber, false
     );
 
     // Update stats
-    ext.pages.query({active: true, lastFocusedWindow: true}, pages =>
+    chrome.tabs.query({active: true, lastFocusedWindow: true}, tabs =>
     {
-      currentPage = pages[0];
+      currentPage = ext.createPageObject(tabs[0]);
       updateStats();
 
       FilterNotifier.on("filter.hitCount", updateStats);
 
       document.getElementById("stats-container").removeAttribute("hidden");
     });
   }
 
   function onUnload()
   {
     FilterNotifier.off("filter.hitCount", updateStats);
   }
 
   function updateStats()
   {
     let statsPage = document.getElementById("stats-page");
     let blockedPage = getBlockedPerPage(currentPage).toLocaleString();
     i18n.setElementText(statsPage, "stats_label_page", [blockedPage]);
 
     let statsTotal = document.getElementById("stats-total");
-    let blockedTotal = Prefs.blocked_total.toLocaleString();
-    i18n.setElementText(statsTotal, "stats_label_total", [blockedTotal]);
+    ext.prefs.get("blocked_total", blockedTotal =>
+    {
+      i18n.setElementText(statsTotal, "stats_label_total",
+                          [blockedTotal.toLocaleString()]);
+    });
   }
 
   function share(ev)
   {
-    // Easter Egg
-    let blocked = Prefs.blocked_total;
-    if (blocked <= 9000 || blocked >= 10000)
-      blocked = blocked.toLocaleString();
-    else
-      blocked = i18n.getMessage("stats_over", (9000).toLocaleString());
+    ext.prefs.get("blocked_total", blockedTotal =>
+    {
+      // Easter Egg
+      if (blockedTotal <= 9000 || blockedTotal >= 10000)
+        blockedTotal = blockedTotal.toLocaleString();
+      else
+        blockedTotal = i18n.getMessage("stats_over", (9000).toLocaleString());
 
-    ext.pages.open(createShareLink(ev.target.dataset.social, blocked));
+      chrome.tabs.create({
+        url: createShareLink(ev.target.dataset.social, blockedTotal)
+      });
+    });
   }
 
   function toggleIconNumber()
   {
-    Prefs.show_statsinicon = !Prefs.show_statsinicon;
-    document.getElementById("show-iconnumber").setAttribute(
-      "aria-checked", Prefs.show_statsinicon
-    );
+    ext.prefs.toggle("show_statsinicon", () =>
+    {
+      ext.prefs.get("show_statsinicon", showStatsInIcon =>

[Manish Jethani, 2017/09/18 02:25:51]

There's no need to call prefs.get now since prefs.toggle already returns the new
value.

+      {
+        document.getElementById("show-iconnumber").setAttribute(
+          "aria-checked", showStatsInIcon
+        );
+      });
+    });
   }
 
   document.addEventListener("DOMContentLoaded", onLoad, false);
   window.addEventListener("unload", onUnload, false);
 }());