Summary: | feColorMatrix saturation is not limited to range 0..1 anymore | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Dirk Schulze <krit> | ||||
Component: | SVG | Assignee: | Dirk Schulze <krit> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | dglazkov, dino, webkit.review.bot, zimmermann | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
URL: | http://dahlström.net/svg/filters/colormatrix/svgimagefiltering.xhtml | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 68469, 26389 | ||||||
Attachments: |
|
Description
Dirk Schulze
2011-10-03 03:20:26 PDT
Created attachment 109464 [details]
Patch
Comment on attachment 109464 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=109464&action=review Patch looks good, r=me. > LayoutTests/svg/filters/feColorMatrix-values.svg:66 > + <text y="340" x="210" style="filter:url(#satrange)">Saturate must </text> > + <text y="370" x="210" style="filter:url(#satrange)">have one value.</text> I don't see this text in the png. Can you remove it and make it a comment? (In reply to comment #2) > (From update of attachment 109464 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=109464&action=review > > Patch looks good, r=me. > > > LayoutTests/svg/filters/feColorMatrix-values.svg:66 > > + <text y="340" x="210" style="filter:url(#satrange)">Saturate must </text> > > + <text y="370" x="210" style="filter:url(#satrange)">have one value.</text> > > I don't see this text in the png. Can you remove it and make it a comment? Thats the sense of this test. You should just see the text in the png, if the test fails. Wasn't my idea, but I like it :) (In reply to comment #3) > (In reply to comment #2) > > (From update of attachment 109464 [details] [details]) > > View in context: https://bugs.webkit.org/attachment.cgi?id=109464&action=review > > > > Patch looks good, r=me. > > > > > LayoutTests/svg/filters/feColorMatrix-values.svg:66 > > > + <text y="340" x="210" style="filter:url(#satrange)">Saturate must </text> > > > + <text y="370" x="210" style="filter:url(#satrange)">have one value.</text> > > > > I don't see this text in the png. Can you remove it and make it a comment? > > Thats the sense of this test. You should just see the text in the png, if the test fails. Wasn't my idea, but I like it :) Aww, I understand, okay just go ahead then :-) Comment on attachment 109464 [details] Patch Attachment 109464 [details] did not pass chromium-ews (chromium-xvfb): Output: http://queues.webkit.org/results/9935104 New failing tests: svg/filters/feColorMatrix-saturate.svg svg/filters/feColorMatrix-values.svg Comment on attachment 109464 [details] Patch Clearing flags on attachment: 109464 Committed r96498: <http://trac.webkit.org/changeset/96498> All reviewed patches have been landed. Closing bug. Committed r96504: <http://trac.webkit.org/changeset/96504> |