Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: background.js

Issue 29337566: Issue 3685 - Remove dead migration code from subscription initilization (Closed)
Patch Set: Created Feb. 23, 2016, 4:25 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: background.js
===================================================================
--- a/background.js
+++ b/background.js
@@ -150,11 +150,6 @@
*/
function addSubscription(prevVersion)
{
- // Make sure to remove "Recommended filters", no longer necessary
- var toRemove = "https://easylist-downloads.adblockplus.org/chrome_supplement.txt";
- if (toRemove in FilterStorage.knownSubscriptions)
- FilterStorage.removeSubscription(FilterStorage.knownSubscriptions[toRemove]);
-
// Add "acceptable ads" subscription for new users
var addAcceptable = !prevVersion;
if (addAcceptable)
@@ -197,19 +192,6 @@
addAcceptable = false;
}
- // Add "anti-adblock messages" subscription for new users and users updating from old ABP versions
- if (!prevVersion || Services.vc.compare(prevVersion, "1.8") < 0)
- {
- var subscription = Subscription.fromURL(Prefs.subscriptions_antiadblockurl);
- if (subscription && !(subscription.url in FilterStorage.knownSubscriptions))
- {
- subscription.disabled = true;
- FilterStorage.addSubscription(subscription);
- if (subscription instanceof DownloadableSubscription && !subscription.lastDownload)
- Synchronizer.execute(subscription);
- }
- }
-
if (!addSubscription && !addAcceptable)
return;
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld