Make ParsePositiveVariant restrict integers too, which affects existing call for -moz-column-count. (Bug 156716) r=bzbarsky sr=roc

layout/style/nsCSSParser.cpp

@@ -3926,7 +3926,12 @@ PRBool CSSParserImpl::ParsePositiveVariant(nsresult& aErrorCode,
         UngetToken();
         return PR_FALSE; 
       } 
-    } 
+    } else if (aValue.GetUnit() == eCSSUnit_Integer) {
+      if (aValue.GetIntValue() < 0) { 
+        UngetToken();
+        return PR_FALSE; 
+      } 
+    }
     return PR_TRUE; 
   } 
   return PR_FALSE; 

layout/style/test/property_database.js

@@ -341,7 +341,7 @@ var gCSSProperties = {
 		initial_values: [ "auto" ],
 		other_values: [ "1", "0", "17" ],
 		invalid_values: [
-			// "-1", unclear: see http://lists.w3.org/Archives/Public/www-style/2007Apr/0030
+			"-1", //unclear: see http://lists.w3.org/Archives/Public/www-style/2007Apr/0030
 			"3px"
 		]
 	},