Issue 6783 - opt-in to anti circumvention filter list on upgrade

lib/subscriptionInit.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
@@ skipping 194 matching lines @@
       let antiAdblockSubscription = Subscription.fromURL(
         Prefs.subscriptions_antiadblockurl
       );
       antiAdblockSubscription.disabled = true;
       subscriptions.push(antiAdblockSubscription);
     }
   }
 
   // Add default ad blocking subscriptions (e.g. EasyList, Anti-Circumvention)
   let addDefaultSubscription = shouldAddDefaultSubscriptions();
-  if (addDefaultSubscription || !Prefs.subscriptions_checkedanticv)
+  if (addDefaultSubscription || !Prefs.subscriptions_addedanticv)
   {
     return fetch("subscriptions.xml")
       .then(response => response.text())
       .then(text =>
       {
         let doc = new DOMParser().parseFromString(text, "application/xml");
         let nodes = doc.getElementsByTagName("subscription");
 
         let defaultSubscriptions = chooseFilterSubscriptions(nodes);
         if (defaultSubscriptions)
@@ skipping 13 matching lines @@
               if (!addDefaultSubscription && type != "circumvention")
                 continue;
 
               let subscription = Subscription.fromURL(url);
               subscription.disabled = false;
               subscription.title = node.getAttribute("title");
               subscription.homepage = node.getAttribute("homepage");
               subscription.type = type;
               subscriptions.push(subscription);
               if (subscription.type == "circumvention")
-                Prefs.subscriptions_checkedanticv = true;

[kzar, 2018/07/16 16:37:47]

Nit: Maybe "added" instead of "checked" in the preference name? To me that would
be more illustrative of this logic.

[hub, 2018/07/16 19:07:35]

I use "checked" because the code checked it and took action. I don't mind
changing the key "subscriptions_addedanticv" if you prefer.

[kzar, 2018/07/17 10:32:32]

Up to you.

[hub, 2018/07/17 14:29:28]

Changed it.

+                Prefs.subscriptions_addedanticv = true;
             }
           }
         }
 
         return subscriptions;
       });
   }
 
   return subscriptions;
 }
@@ skipping 73 matching lines @@
  *
  * @param {function} callback
  */
 exports.setSubscriptionsCallback = callback =>
 {
   subscriptionsCallback = callback;
 };
 
 // Exports for tests only
 exports.chooseFilterSubscriptions = chooseFilterSubscriptions;