Issue 3654 - Properly split up selectors when applying CSS property filters

chrome/content/cssProperties.js

 function splitSelector(selector)

[Sebastian Noack, 2016/02/15 17:19:08]

This logic has been copied (with some minor changes) from
adblockpluschrome/include.perload.js. After the dependency update we should
reuse splitSelector() there.

[kzar, 2016/02/15 17:30:14]

Acknowledged. (Mind filing an issue for that?)

[Sebastian Noack, 2016/02/15 17:49:31]

I attached the patch the same issue:
https://codereview.adblockplus.org/29336409/

 {
   if (selector.indexOf(",") == -1)
     return [selector];
 
   var selectors = [];
   var start = 0;
   var level = 0;
   var sep = "";
 
   for (var i = 0; i < selector.length; i++)
@@ skipping 21 matching lines @@
   }
 
   selectors.push(selector.substring(start));
   return selectors;
 }
 
 function CSSPropertyFilters(window, addSelectorsFunc) {
   this.window = window;
   this.addSelectorsFunc = addSelectorsFunc;
 }
+
 CSSPropertyFilters.prototype = {
   stringifyStyle: function(style)
   {
     var styles = [];
     for (var i = 0; i < style.length; i++)
     {
       var property = style.item(i);
       var value    = style.getPropertyValue(property);
       var priority = style.getPropertyPriority(property);
       styles.push(property + ": " + value + (priority ? " !" + priority : "") + ";");
@@ skipping 66 matching lines @@
   apply: function()
   {
     if (this.patterns.length > 0)
     {
       var document = this.window.document;
       this.addSelectors(document.styleSheets);
       document.addEventListener("load", this.onLoad.bind(this), true);
     }
   }
 };