Issue 3670 - Make rules case sensitive where possible

lib/contentBlockerList.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 47 matching lines @@
   let included = [];
   let excluded = [];
   let rules = [];
 
   parseDomains(filter.domains, included, excluded);
 
   if (excluded.length == 0 && !(filter.selector in elemhideSelectorExceptions))
     return {matchDomains: included.map(matchDomain), selector: filter.selector};
 }
 
-// Convert the "regexpSource" part of a filter's text to a regular expression,

[Sebastian Noack, 2016/02/25 02:28:35]

Sorry for commenting after LGTM. Just one more nit: While I don't consider
documentation for internal APIs mandatory, if you comment private functions,
please still use JSDoc syntax.

[kzar, 2016/02/25 15:22:28]

Done.

-// also deciding if the expression can safely be converted to and matched as
-// lowercase.
+/**
+ * Convert the given filter "regexpSource" string into a regular expression.
+ * (Also deciding if the regular expression can be safely converted to and
+ *  matched as lower case or not.)
+ *
+ * @param   {string} text regexpSource property of a filter
+ * @returns {object} An object containing a regular expression string and a bool
+ *                   indicating if the filter can be safely matched as lower
+ *                   case: {regexp: "...", caseSenstive: true/false}
+ */
 function toRegExp(text)
 {
   let result = [];
   let lastIndex = text.length - 1;
   let hostnameStarted = false;
   let hostnameFinished = false;
   let caseSensitive = false;
 
   for (let i = 0; i < text.length; i++)
   {
@@ skipping 339 matching lines @@
 
   for (let filter of this.elemhideExceptions)
     addRule(convertFilter(filter, "ignore-previous-rules", false));
   for (let filter of this.requestFilters)
     addRule(convertFilter(filter, "block", true));
   for (let filter of this.requestExceptions)
     addRule(convertFilter(filter, "ignore-previous-rules", true));
 
   return rules;
 };