Issue 4658 - Incorporate element hiding emulation filters

chrome/content/ui/filters-backup.js

Index: chrome/content/ui/filters-backup.js
===================================================================
--- a/chrome/content/ui/filters-backup.js
+++ b/chrome/content/ui/filters-backup.js
@@ -312,25 +312,25 @@
         list.push(filter.text);
 
         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";
         }
+
+        if (filter instanceof ElemHideEmulationFilter && Services.vc.compare(minVersion, "2.9") < 0)
+          minVersion = "2.9";

[Wladimir Palant, 2016/12/13 14:44:27]

No, as far as our users are concerned the syntax is supported starting with
2.7.3 - different class name changes nothing. Once the :has syntax is added we
will need to bump the version here, any only for filters actually using :has.

[Felix Dahlke, 2016/12/13 15:55:39]

Acknowledged. (Oh boy, sounds like we'll have to reintroduce the feature bit
mask after all?)

[Wladimir Palant, 2016/12/13 16:33:03]

No, this feature is purely informative - meaning that looking for ":has(" in the
selector will be sufficient.

       }
     }
     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);