HASH_IDENT needs to be handled as well -- this fixes w3c#59

ylafon · ylafon · commit d709e7cb4ff4 · 2016-09-19T15:50:21.000+02:00
diff --git a/org/w3c/css/properties/svg/CssFill.java b/org/w3c/css/properties/svg/CssFill.java
@@ -9,6 +9,7 @@
 import org.w3c.css.properties.css.CssProperty;
 import org.w3c.css.util.ApplContext;
 import org.w3c.css.util.InvalidParamException;
+import org.w3c.css.values.CssColor;
 import org.w3c.css.values.CssExpression;
 import org.w3c.css.values.CssFunction;
 import org.w3c.css.values.CssIdent;
@@ -132,6 +133,17 @@ static CssValue parsePaint(ApplContext ac, CssExpression expression,
 					}
 					gotColor = true;
 					break;
+				case CssTypes.CSS_HASH_IDENT:
+					if (gotColor) {
+						throw new InvalidParamException("value",
+								val.toString(),
+								property.getPropertyName(), ac);
+					}
+					CssColor c = new CssColor();
+					c.setShortRGBColor(val.toString(), ac);
+					gotColor = true;
+					values.add(c);
+					break;
 				case CssTypes.CSS_COLOR:
 					if (gotColor) {
 						throw new InvalidParamException("value",
diff --git a/org/w3c/css/properties/svg/CssStopColor.java b/org/w3c/css/properties/svg/CssStopColor.java
@@ -9,6 +9,7 @@
 import org.w3c.css.properties.css.CssProperty;
 import org.w3c.css.util.ApplContext;
 import org.w3c.css.util.InvalidParamException;
+import org.w3c.css.values.CssColor;
 import org.w3c.css.values.CssExpression;
 import org.w3c.css.values.CssFunction;
 import org.w3c.css.values.CssIdent;
@@ -119,6 +120,17 @@ static CssValue parseColor(ApplContext ac, CssExpression expression,
 					}
 					gotColor = true;
 					break;
+				case CssTypes.CSS_HASH_IDENT:
+					if (gotColor) {
+						throw new InvalidParamException("value",
+								val.toString(),
+								property.getPropertyName(), ac);
+					}
+					CssColor c = new CssColor();
+					c.setShortRGBColor(val.toString(), ac);
+					gotColor = true;
+					values.add(c);
+					break;
 				case CssTypes.CSS_COLOR:
 					if (gotColor) {
 						throw new InvalidParamException("value",
