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

Unified Diff: chrome/content/ui/filters-backup.js

Issue 29357587: Issue 4538 - When exporting filters, mark $websocket filters as requiring Adblock Plus 2.8 (Closed) Base URL: https://hg.adblockplus.org/adblockplus
Patch Set: Created Oct. 17, 2016, 11:18 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: chrome/content/ui/filters-backup.js
===================================================================
--- a/chrome/content/ui/filters-backup.js
+++ b/chrome/content/ui/filters-backup.js
@@ -314,16 +314,23 @@ var Backup =
if (filter instanceof ElemHideException && Services.vc.compare(minVersion, "2.1") < 0)
minVersion = "2.1";
if (filter instanceof RegExpFilter && filter.contentType & (RegExpFilter.typeMap.GENERICHIDE | RegExpFilter.typeMap.GENERICBLOCK) && Services.vc.compare(minVersion, "2.6.12") < 0)
minVersion = "2.6.12";
if (filter instanceof CSSPropertyFilter && Services.vc.compare(minVersion, "2.7.3") < 0)
minVersion = "2.7.3";
+
+ if (filter instanceof RegExpFilter &&
+ (filter.contentType & RegExpFilter.typeMap.WEBSOCKET) &&
+ Services.vc.compare(minVersion, "2.8"))
+ {
+ minVersion = "2.8";
+ }
}
}
list.unshift("[Adblock Plus " + minVersion + "]");
// Insert checksum. Have to add an empty line to the end of the list to
// account for the trailing newline in the file.
list.push("");
let checksum = Utils.generateChecksum(list);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld