LayoutTests/svg/repaint/fecomponenttransfer-in1-change.svg - Issue 1046673006: Add svgAttributeChanged function to SVGFEComponentTransferElement class

Unified Diff: LayoutTests/svg/repaint/fecomponenttransfer-in1-change.svg

Issue 1046673006: Add svgAttributeChanged function to SVGFEComponentTransferElement class (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: modified testcases. Created 5 years, 9 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« LayoutTests/svg/dynamic-updates/script-tests/SVGFEComponentTransferElement-dom-amplitude-attr.js ('K') | « LayoutTests/svg/dynamic-updates/script-tests/SVGFEComponentTransferElement-svgdom-type-prop.js ('k') | LayoutTests/svg/repaint/fecomponenttransfer-in1-change-expected.png » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/svg/repaint/fecomponenttransfer-in1-change.svg

diff --git a/LayoutTests/svg/repaint/fecomponenttransfer-in1-change.svg b/LayoutTests/svg/repaint/fecomponenttransfer-in1-change.svg

new file mode 100644

index 0000000000000000000000000000000000000000..95e5069ee7e631442d27d4fbefd733ec422e98e8

--- /dev/null

+++ b/LayoutTests/svg/repaint/fecomponenttransfer-in1-change.svg

@@ -0,0 +1,20 @@

+<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink"

+ width="300" height="300" onload="runRepaintAndPixelTest()">

fs 2015/04/01 16:21:41 Should be no need to make a pixel test for this -

+<script xlink:href="../../fast/repaint/resources/text-based-repaint.js" />

fs 2015/04/01 16:21:41 If you put the test in an HTML wrapper you can dro

+<script><![CDATA[

+function repaintTest() {

+ var fe = document.getElementById("fe");

+ fe.in1.baseVal = "SourceGraphic";

+]]></script>

+<linearGradient id="MyGradient" gradientUnits="userSpaceOnUse" x1="10" y1="0" x2="590" y2="0">

fs 2015/04/01 16:21:41 There should be no need for a gradient here. Just

+ <stop offset="0" stop-color="#00ff00"/>

+ <stop offset="1" stop-color="#000000"/>

+</linearGradient>

+<filter id="SourceGraphic" x="0%" y="0%" width="100%" height="100%">

fs 2015/04/01 16:21:41 Just call it something simple like 'f' ('SourceGra

+ <feComponentTransfer id="fe" in="SourceAlpha">

+ <feFuncG type="gamma" amplitude="1" exponent="0.5" offset="0"/>

fs 2015/04/01 16:21:41 I think you could just use type="identity" here.

+ </feComponentTransfer>

+</filter>

+<rect x="0" y="0" width="100" height="100" fill="url(#MyGradient)" filter="url(#SourceGraphic)"/>

fs 2015/04/01 16:21:41 Drop x="0" and y="0" (they're the default values)

+</svg>