simplify updating setting showing blocked resources container in popup

EFForg · May 6, 2021 · 6408edb · 6408edb
1 parent cad1393
commit 6408edb
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 14 deletions.
diff --git a/src/js/popup.js b/src/js/popup.js
@@ -467,13 +467,15 @@ function toggleBlockedResourcesHandler() {
     $("#collapse-blocked-resources, #blockedResources").show();
     $("#expand-blocked-resources").hide();
     chrome.runtime.sendMessage({
-      type: "showTrackingDomainsSection"
+      type: "updateSettings",
+      data: { showExpandedTrackingSection: true }
     });
   } else {
     $("#collapse-blocked-resources, #blockedResources").hide();
     $("#expand-blocked-resources").show();
     chrome.runtime.sendMessage({
-      type: "hideTrackingDomainsSection"
+      type: "updateSettings",
+      data: { showExpandedTrackingSection: false }
     });
   }
 }

diff --git a/src/js/webrequest.js b/src/js/webrequest.js
@@ -1128,18 +1128,6 @@ function dispatcher(request, sender, sendResponse) {
     break;
   }
 
-  case "showTrackingDomainsSection": {
-    badger.getSettings().setItem("showExpandedTrackingSection", true);
-    sendResponse();
-    break;
-  }
-
-  case "hideTrackingDomainsSection": {
-    badger.getSettings().setItem("showExpandedTrackingSection", false);
-    sendResponse();
-    break;
-  }
-
   case "downloadCloud": {
     chrome.storage.sync.get("disabledSites", function (store) {
       if (chrome.runtime.lastError) {