Issue 3870 - Rewrite legacy options page to use async messages

options.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 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 54 matching lines @@
 var getPref = wrapper({type: "prefs.get"}, "key");
 var togglePref = wrapper({type: "prefs.toggle"}, "key");
 var getSubscriptions = wrapper({type: "subscriptions.get"},
                                "downloadable", "special");
 var removeSubscription = wrapper({type: "subscriptions.remove"}, "url");
 var addSubscription = wrapper({type: "subscriptions.add"},
                                 "url", "title", "homepage");
 var toggleSubscription = wrapper({type: "subscriptions.toggle"},
                                  "url", "keepInstalled");
 var updateSubscription = wrapper({type: "subscriptions.update"}, "url");
-var isSubscriptionDownloading = wrapper({type: "subscriptions.isDownloading"},
-                                        "url");
 var importRawFilters = wrapper({type: "filters.importRaw"},
                                "text", "removeExisting");
 var addFilter = wrapper({type: "filters.add"}, "text");
 var getFilters = wrapper({type: "filters.get"}, "subscriptionUrl");
 var removeFilter = wrapper({type: "filters.remove"}, "text");
 
 var i18n = ext.i18n;
 var whitelistedDomainRegexp = /^@@\|\|([^\/:]+)\^\$document$/;
 var delayedSubscriptionSelection = null;
 
+var acceptableAdsUrl;
+
 // Loads options and sets UI elements accordingly
 function loadOptions()
 {
   // Set page title to i18n version of "Adblock Plus Options"
   document.title = i18n.getMessage("options");
 
   // Set links
   getPref("subscriptions_exceptionsurl", function(url)
   {
-    $("#acceptableAdsLink").attr("href", url);
+    acceptableAdsUrl = url;
+    $("#acceptableAdsLink").attr("href", acceptableAdsUrl);
   });
   getDocLink("acceptable_ads", function(url)
   {
     $("#acceptableAdsDocs").attr("href", url);
   });
   getDocLink("filterdoc", function(url)
   {
     setLinks("filter-must-follow-syntax", url);
   });
   getInfo("application", function(application)
@@ skipping 48 matching lines @@
   });
   getPref("notifications_showui", function(notifications_showui)
   {
     if (!notifications_showui)
       document.getElementById("shouldShowNotificationsContainer").hidden = true;
   });
 
   // Register listeners in the background message responder
   ext.backgroundPage.sendMessage({
     type: "app.listen",
-    filter: ["addSubscription", "switchToOptionsSection"]
+    filter: ["addSubscription", "focusSection"]
   });
   ext.backgroundPage.sendMessage(
   {
     type: "filters.listen",
     filter: ["added", "loaded", "removed"]
   });
   ext.backgroundPage.sendMessage(
   {
     type: "prefs.listen",
     filter: ["notifications_ignoredcategories", "notifications_showui",
-        "safari_contentblocker", "show_devtools_panel",
-        "shouldShowBlockElementMenu"]
+             "safari_contentblocker", "show_devtools_panel",
+             "shouldShowBlockElementMenu"]
   });
   ext.backgroundPage.sendMessage(
   {
     type: "subscriptions.listen",
     filter: ["added", "disabled", "homepage", "lastDownload", "removed",
-             "title"]
+             "title", "downloadStatus", "downloading"]
   });
 
   // Load recommended subscriptions
   loadRecommendations();
 
   // Show user's filters
   reloadFilters();
 }
 $(loadOptions);
 
 // Reloads the displayed subscriptions and filters
 function reloadFilters()
 {
   // Load user filter URLs
   var container = document.getElementById("filterLists");
   while (container.lastChild)
     container.removeChild(container.lastChild);
 
-  getPref("subscriptions_exceptionsurl", function(acceptableAdsUrl)
-  {
-    getSubscriptions(true, false, function(subscriptions)
-    {
-      for (var i = 0; i < subscriptions.length; i++)
-      {
-        var subscription = subscriptions[i];
-        if (subscription.url == acceptableAdsUrl)
-          $("#acceptableAds").prop("checked", !subscription.disabled);
-        else
-          addSubscriptionEntry(subscription);
-      }
-    });
+  getSubscriptions(true, false, function(subscriptions)
+  {
+    for (var i = 0; i < subscriptions.length; i++)
+    {
+      var subscription = subscriptions[i];
+      if (subscription.url == acceptableAdsUrl)
+        $("#acceptableAds").prop("checked", !subscription.disabled);
+      else
+        addSubscriptionEntry(subscription);
+    }
   });
 
   // User-entered filters
   getSubscriptions(false, true, function(subscriptions)
   {
     clearListBox("userFiltersBox");
     clearListBox("excludedDomainsBox");
 
     for (var i = 0; i < subscriptions.length; i++)
       getFilters(subscriptions[i].url, function(filters)
@@ skipping 162 matching lines @@
     addSubscription(url, title, null);
   }
 
   $("#addSubscriptionContainer").hide();
   $("#customSubscriptionContainer").hide();
   $("#addSubscriptionButton").show();
 }
 
 function toggleAcceptableAds()
 {
-  getPref("subscriptions_exceptionsurl", function(url)
-  {
-    toggleSubscription(url, true);
-  });
+  toggleSubscription(acceptableAdsUrl, true);
 }
 
 function findSubscriptionElement(subscription)
 {
   var children = document.getElementById("filterLists").childNodes;
   for (var i = 0; i < children.length; i++)
     if (children[i]._subscription.url == subscription.url)
       return children[i];
   return null;
 }
@@ skipping 12 matching lines @@
     title.href = subscription.homepage;
   else
     title.href = subscription.url;
 
   var enabled = element.getElementsByClassName("subscriptionEnabled")[0];
   enabled.checked = !subscription.disabled;
 
   var lastUpdate = element.getElementsByClassName("subscriptionUpdate")[0];
   lastUpdate.classList.remove("error");
 
-  if (subscription.downloadStatus &&
-      subscription.downloadStatus != "synchronize_ok")
+  var downloadStatus = subscription.downloadStatus;
+  if (subscription.isDownloading)
+  {
+    lastUpdate.textContent = i18n.getMessage("filters_subscription_lastDownload_inProgress");
+  }
+  else if (downloadStatus && downloadStatus != "synchronize_ok")
   {
      var map =
      {
        "synchronize_invalid_url": "filters_subscription_lastDownload_invalidURL",
        "synchronize_connection_error": "filters_subscription_lastDownload_connectionError",
        "synchronize_invalid_data": "filters_subscription_lastDownload_invalidData",
        "synchronize_checksum_mismatch": "filters_subscription_lastDownload_checksumMismatch"
      };
-     if (subscription.downloadStatus in map)
-       lastUpdate.textContent = i18n.getMessage(map[subscription.downloadStatus]);
+     if (downloadStatus in map)
+       lastUpdate.textContent = i18n.getMessage(map[downloadStatus]);
      else
-       lastUpdate.textContent = subscription.downloadStatus;
+       lastUpdate.textContent = downloadStatus;
      lastUpdate.classList.add("error");
   }
   else if (subscription.lastDownload > 0)
   {
     var timeDate = i18n_timeDateStrings(subscription.lastDownload * 1000);
     var messageID = (timeDate[1] ? "last_updated_at" : "last_updated_at_today");
     lastUpdate.textContent = i18n.getMessage(messageID, timeDate);
   }
 }
 
 function onSubscriptionMessage(action, subscription)
 {
+  var element = findSubscriptionElement(subscription);
+
   switch (action)
   {
-    case "title":
     case "disabled":
+    case "downloading":
+    case "downloadStatus":
     case "homepage":
     case "lastDownload":
-    case "downloadStatus":
-      var element = findSubscriptionElement(subscription);
+    case "title":
       if (element)
         updateSubscriptionInfo(element, subscription);
       break;
     case "added":
-      getPref("subscriptions_exceptionsurl", function(acceptableAdsUrl)
-      {
-        if (subscription.url == acceptableAdsUrl)
-          $("#acceptableAds").prop("checked", true);
-        else if (!findSubscriptionElement(subscription))
-          addSubscriptionEntry(subscription);
-      });
+      if (subscription.url == acceptableAdsUrl)
+        $("#acceptableAds").prop("checked", true);
+      else if (!element)
+        addSubscriptionEntry(subscription);
       break;
     case "removed":
-      getPref("subscriptions_exceptionsurl", function(acceptableAdsUrl)
-      {
-        if (subscription.url == acceptableAdsUrl)
-        {
-          $("#acceptableAds").prop("checked", false);
-        }
-        else
-        {
-          var element = findSubscriptionElement(subscription);
-          if (element)
-            element.parentNode.removeChild(element);
-        }
-      });
+      if (subscription.url == acceptableAdsUrl)
+        $("#acceptableAds").prop("checked", false);
+      else if (element)
+        element.parentNode.removeChild(element);
       break;
   }
 }
 
 function onPrefMessage(key, value)
 {
   switch (key)
   {
     case "notifications_showui":
       document.getElementById("shouldShowNotificationsContainer").hidden = !value;
@@ skipping 145 matching lines @@
     else
       $("#rawFilters").hide();
   });
 }
 
 // Called when user explicitly requests filter list updates
 function updateFilterLists()
 {
   // Without the URL parameter this will update all subscriptions
   updateSubscription();
-
-  // Display a message for any subscriptions that are currently being downloaded
-  getSubscriptions(true, false, function(subscriptions)
-  {
-    var inProgressMessage = i18n.getMessage(
-      "filters_subscription_lastDownload_inProgress"
-    );
-
-    for (var i = 0; i < subscriptions.length; i += 1)
-    {
-      var element = findSubscriptionElement(subscriptions[i]);
-      if (element)
-      {
-        var label = element.getElementsByClassName("subscriptionUpdate")[0];
-        isSubscriptionDownloading(subscriptions[i].url,
-          function(label, isDownloading)
-          {
-            if (isDownloading)
-              label.textContent = inProgressMessage;
-          }.bind(undefined, label)
-        );
-      }
-    }
-  });
 }
 
 // Adds a subscription entry to the UI.
 function addSubscriptionEntry(subscription)
 {
   var template = document.getElementById("subscriptionTemplate");
   var element = template.cloneNode(true);
   element.removeAttribute("id");
   element._subscription = subscription;
 
@@ skipping 52 matching lines @@
 {
   switch (message.type)
   {
     case "app.respond":
       switch (message.action)
       {
         case "addSubscription":
           var subscription = message.args[0];
           startSubscriptionSelection(subscription.title, subscription.url);
           break;
-        case "switchToOptionsSection":
+        case "focusSection":
           var tabs = document.getElementsByClassName("ui-tabs-panel");
           for (var i = 0; i < tabs.length; i++)
           {
             var found = tabs[i].querySelector(
               "[data-section='" + message.args[0] + "']"
             );
             if (!found)
               continue;
 
             var previous = document.getElementsByClassName("focused");
@@ skipping 11 matching lines @@
       onFilterMessage(message.action, message.args[0]);
       break;
     case "prefs.respond":
       onPrefMessage(message.action, message.args[0]);
       break;
     case "subscriptions.respond":
       onSubscriptionMessage(message.action, message.args[0]);
       break;
   }
 });