Fix bad string comparison preventing 'none' and 'initial' values of 'counter-increment' and 'counter-reset' from being parsed correctly. b=379555 r+sr=dbaron Patch by Daniel Holbert <dholbert@mozilla.com>.

layout/style/nsCSSParser.cpp

@@ -5455,7 +5455,7 @@ PRBool CSSParserImpl::ParseCounterData(nsresult& aErrorCode,
   for (const SingleCounterPropValue *sv = singleValues,
            *sv_end = singleValues + NS_ARRAY_LENGTH(singleValues);
        sv != sv_end; ++sv) {
-    if (ident->LowerCaseEqualsLiteral(sv->str)) {
+    if (ident->LowerCaseEqualsASCII(sv->str)) {
       if (ExpectEndProperty(aErrorCode, PR_TRUE)) {
         nsCSSCounterData* dataHead = new nsCSSCounterData();
         if (!dataHead) {

layout/style/test/test_inherit_computation.html

@@ -75,10 +75,6 @@ var gBrokenInherit = {
   "-moz-border-left-colors": true,
   "-moz-border-right-colors": true,
   "-moz-border-top-colors": true,
-
-  // Parses inherit as a counter name
-  "counter-increment": true,
-  "counter-reset": true,
 };
 
 function xfail_inherit(property, matching_initial) {

layout/style/test/test_value_computation.html

@@ -107,10 +107,6 @@ var gBadComputed = {
   "clip": [ "rect(auto,auto,auto,auto)" ],
 
   // This seems really broken!
-  "counter-increment": [ "none" ],
-  "counter-reset": [ "none" ],
-
-  // So does this.
   "marker-offset": [ "auto" ],
 
   // No idea what's happening for these two either