Range check must be done for percentages as well (See #201)

Author: ylafon

org/w3c/css/values/RGB.java

@@ -18,6 +18,7 @@
 import org.w3c.css.util.Util;
 
 public class RGB {
+    static final String functionname = "rgb";
 
     private String output = null;
     private boolean percent = false;
@@ -48,6 +49,22 @@ public final void setRed(ApplContext ac, CssValue val)
                         nv = nb;
                     }
                 }
+            } else if (val.getType() == CssTypes.CSS_PERCENTAGE) {
+                CssCheckableValue v = val.getCheckableValue();
+                if (!v.warnPositiveness(ac, "RGB")) {
+                    CssNumber nb = new CssNumber();
+                    nb.setIntValue(0);
+                    nv = nb;
+                }
+                if (val.getRawType() == CssTypes.CSS_NUMBER) {
+                    float p = ((CssNumber) val).getValue();
+                    if (p > 100.) {
+                        ac.getFrame().addWarning("out-of-range", Util.displayFloat(p));
+                        CssNumber nb = new CssNumber();
+                        nb.setIntValue(100);
+                        nv = nb;
+                    }
+                }
             }
         }
         vr = nv;
@@ -77,6 +94,22 @@ public final void setGreen(ApplContext ac, CssValue val)
                         nv = nb;
                     }
                 }
+            } else if (val.getType() == CssTypes.CSS_PERCENTAGE) {
+                CssCheckableValue v = val.getCheckableValue();
+                if (!v.warnPositiveness(ac, "RGB")) {
+                    CssNumber nb = new CssNumber();
+                    nb.setIntValue(0);
+                    nv = nb;
+                }
+                if (val.getRawType() == CssTypes.CSS_NUMBER) {
+                    float p = ((CssNumber) val).getValue();
+                    if (p > 100.) {
+                        ac.getFrame().addWarning("out-of-range", Util.displayFloat(p));
+                        CssNumber nb = new CssNumber();
+                        nb.setIntValue(100);
+                        nv = nb;
+                    }
+                }
             }
         }
         vg = nv;
@@ -107,6 +140,22 @@ public final void setBlue(ApplContext ac, CssValue val)
                         nv = nb;
                     }
                 }
+            } else if (val.getType() == CssTypes.CSS_PERCENTAGE) {
+                CssCheckableValue v = val.getCheckableValue();
+                if (!v.warnPositiveness(ac, "RGB")) {
+                    CssNumber nb = new CssNumber();
+                    nb.setIntValue(0);
+                    nv = nb;
+                }
+                if (val.getRawType() == CssTypes.CSS_NUMBER) {
+                    float p = ((CssNumber) val).getValue();
+                    if (p > 100.) {
+                        ac.getFrame().addWarning("out-of-range", Util.displayFloat(p));
+                        CssNumber nb = new CssNumber();
+                        nb.setIntValue(100);
+                        nv = nb;
+                    }
+                }
             }
         }
         vb = nv;
@@ -168,7 +217,7 @@ protected void setRepresentationString(String s) {
      */
     public String toString() {
         if (output == null) {
-            StringBuilder sb = new StringBuilder("rgb(");
+            StringBuilder sb = new StringBuilder(functionname).append('(');
             if (isPercent()) {
                 sb.append(vr).append("%, ");
                 sb.append(vg).append("%, ");