Issue 3880 - Improve behavior of Safari content blocker option

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 689 matching lines @@
     // Advanced tab
     var tweaks = document.querySelectorAll("#tweaks li[data-pref]");
     tweaks = Array.prototype.map.call(tweaks, function(checkbox)
     {
       return checkbox.getAttribute("data-pref");
     });
     tweaks.forEach(function(key)
     {
       getPref(key, function(value)
       {
-        onPrefMessage(key, value);
+        onPrefMessage(key, value, true);
       });
     });
     ext.backgroundPage.sendMessage(
     {
       type: "app.get",
       what: "features"
     },
     function(features)
     {
       hidePref("show_devtools_panel", !features.devToolsPanel);
@@ skipping 300 matching lines @@
   };
 
   getPref.checks =
   {
     notifications_ignoredcategories: function(callback)
     {
       getPref("notifications_showui", callback);
     }
   };
 
-  function onPrefMessage(key, value)
-  {
+  function onPrefMessage(key, value, initial)
+  {
+    switch (key)
+    {
+      case "notifications_ignoredcategories":
+        value = value.indexOf("*") == -1;
+        break;
+
+      case "notifications_showui":
+        hidePref("notifications_ignoredcategories", !value);
+        break;
+
+      case "safari_contentblocker":
+        E("restart-safari").setAttribute("aria-hidden", value || initial);
+        break;
+    }
+
     var checkbox = document.querySelector("[data-pref='" + key + "'] button[role='checkbox']");
     if (checkbox)
-    {
-      if (key == "notifications_ignoredcategories")
-        value = (value.indexOf("*") == -1);
       checkbox.setAttribute("aria-checked", value);
-    }
-    else if (key == "notifications_showui")
-      hidePref("notifications_ignoredcategories", !value);
   }
 
   function onShareLinkClick(e)
   {
     e.preventDefault();
 
     getDocLink("share-general", function(link)
     {
       openSharePopup(link);
     });
@@ skipping 28 matching lines @@
 
     for (var i = 0; i < shareResources.length; i++)
       checkShareResource(shareResources[i], onResult);
   }
 
   ext.onMessage.addListener(function(message)
   {
     switch (message.type)
     {
       case "app.respond":
-        switch (message.action)
-        {
-          case "addSubscription":
-            var subscription = message.args[0];
-            var dialog = E("dialog-content-predefined");
-            dialog.querySelector("h3").textContent = subscription.title || "";
-            dialog.querySelector(".url").textContent = subscription.url;
-            openDialog("predefined");
-            break;
-          case "safariRestartRequired":
-            E("restart-safari").setAttribute("aria-hidden", !message.args[0]);
-            break;
+        if (message.action == "addSubscription")
+        {
+          var subscription = message.args[0];
+          var dialog = E("dialog-content-predefined");
+          dialog.querySelector("h3").textContent = subscription.title || "";
+          dialog.querySelector(".url").textContent = subscription.url;
+          openDialog("predefined");
         }
         break;
       case "filters.respond":
         onFilterMessage(message.action, message.args[0]);
         break;
       case "prefs.respond":
-        onPrefMessage(message.action, message.args[0]);
+        onPrefMessage(message.action, message.args[0], false);
         break;
       case "subscriptions.respond":
         onSubscriptionMessage(message.action, message.args[0]);
         break;
     }
   });
 
   ext.backgroundPage.sendMessage(
   {
     type: "app.listen",
-    filter: ["addSubscription", "safariRestartRequired"]
+    filter: ["addSubscription"]
   });
   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"]
   });
   ext.backgroundPage.sendMessage(
   {
     type: "subscriptions.listen",
     filter: ["added", "disabled", "homepage", "lastDownload", "removed",
         "title"]
   });
 
   window.addEventListener("DOMContentLoaded", onDOMLoaded, false);
 })();