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

Unified Diff: background.js

Issue 8684064: Remove "Recommended filters" subscriptions (Closed)
Patch Set: Created Oct. 26, 2012, 11:27 a.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
@@ -279,16 +279,21 @@ function importOldData()
}
/**
* This function is called on an extension update. It will add the default
* filter subscription if necessary.
*/
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 and users updating from old ABP versions
var addAcceptable = (!prevVersion || Services.vc.compare(prevVersion, "2.1") < 0);
if (addAcceptable)
{
addAcceptable = !FilterStorage.subscriptions.some(function(subscription)
{
return subscription.url == Prefs.subscriptions_exceptionsurl;
});
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld