Oper doesn't limit the range of values for saturation from 0 to 1. The limitation was also removed from the new filter effects 1.0 specification.
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>