Bug 71335
| Summary: | feColorMatrix can't parse leading space in the values list | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Tim Horton <thorton> |
| Component: | SVG | Assignee: | Nobody <webkit-unassigned> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | krit, simon.fraser, webkit-bug-importer, zimmermann |
| Priority: | P2 | Keywords: | InRadar |
| Version: | 528+ (Nightly build) | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| URL: | http://granite.sru.edu/~ddailey/svg/feComponentTransfer2.svg | ||
Tim Horton
http://granite.sru.edu/~ddailey/svg/feComponentTransfer2.svg
The leading space in the values attribute of <feColorMatrix> is causing the filter not to draw.
There's a marginally related bug where it looked like the spec doesn't allow leading whitespace, but I'm not sure it applies here: https://bugs.webkit.org/show_bug.cgi?id=62546
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Radar WebKit Bug Importer
<rdar://problem/10380036>
Dirk Schulze
Do you mean kernelMatrix? That is a really good question. I'd expect the following rule counts, even if it is not specified in SVG Filters directly:
<list-of-Ts>
(Where T is a type other than <string> and <family-name>.) A list consists of a separated sequence of values. Unless explicitly described differently, lists within SVG's XML attributes can be either comma-separated, with optional white space before or after the comma, or white space-separated.
White space in lists is defined as one or more of the following consecutive characters: "space" (U+0020), "tab" (U+0009), "line feed" (U+000A), "carriage return" (U+000D) and "form-feed" (U+000C).
The following is a template for an EBNF grammar describing the <list-of-Ts> syntax:
list-of-Ts ::= T
| T comma-wsp list-of-Ts
comma-wsp ::= (wsp+ ","? wsp*) | ("," wsp*)
wsp ::= (#x20 | #x9 | #xD | #xA)
Within the SVG DOM, values of a <list-of-Ts> type are represented by an interface specific for the particular type T. For example, a <list-of-lengths> is represented in the SVG DOM using an SVGLengthList or SVGAnimatedLengthList object.
http://www.w3.org/TR/SVG/types.html#DataTypeNumber
So yes, I think leading spaces are not allowed for kernelMatrix as well.
Tim Horton
(In reply to comment #2)
> Do you mean kernelMatrix?
No, I mean the "values" attribute of feColorMatrix.
> So yes, I think leading spaces are not allowed for kernelMatrix as well.
I think this probably applies, regardless. So we should probably close this.