Save list just once when deleting to avoid glitches (#3322)

MozillaReality · May 4, 2020 · 0adafe3 · 0adafe3
1 parent 653ce1c
commit 0adafe3
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/browser/content/TrackingProtectionStore.java b/app/src/common/shared/org/mozilla/vrbrowser/browser/content/TrackingProtectionStore.java
@@ -178,7 +178,17 @@ public void remove(@NonNull SitePermission permission) {
 
     public void removeAll() {
         // We can't use clearExceptionList as that clears also the private browsing exceptions
-        mSitePermissions.forEach(this::remove);
+        mSitePermissions.forEach(permission -> {
+            ContentBlockingException exception = toContentBlockingException(permission);
+            if (exception != null) {
+                mContentBlockingController.removeException(exception);
+            }
+            mListeners.forEach(listener -> listener.onExcludedTrackingProtectionChange(
+                    permission.url,
+                    false,
+                    false));
+        });
+        saveExceptions();
     }
 
     private void saveExceptions() {