LayoutTests/svg/animations/animateMotion-accumulate-2b.svg - Issue 18436004: [SVG] animateMotion accumulate doesn't work properly with rotate: auto / auto-reverse

Side by Side Diff: LayoutTests/svg/animations/animateMotion-accumulate-2b.svg

Issue 18436004: [SVG] animateMotion accumulate doesn't work properly with rotate: auto / auto-reverse (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 7 years, 5 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 unified diff | Download patch

« LayoutTests/svg/animations/animateMotion-accumulate-2a.svg ('K') | « LayoutTests/svg/animations/animateMotion-accumulate-2a-expected.svg ('k') | LayoutTests/svg/animations/animateMotion-accumulate-2b-expected.svg » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
(Empty)
	1 <?xml version="1.0" encoding="iso-8859-1" ?>

	2 <!DOCTYPE svg PUBLIC "-//W3C//DTD SVG Tiny 1.1//EN"

	3 "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11-tiny.dtd">

	4 <svg viewBox="0 0 500 350" onload="loaded()"

	5 xmlns="http://www.w3.org/2000/svg" version="1.1" baseProfile="tiny"

	6 xmlns:xlink="http://www.w3.org/1999/xlink">

	7 <g id="test-body-content" font-family="SVGFreeSansASCII,sans-serif" font-siz e="18">

	8 <rect x="0" y="0" width="480" height="360" fill="#fff" />

	9 <g transform="translate(40,-20) scale(0.45)">

	10 <g>

	11 <circle cx="0" cy="0" r="70" fill="#f00">

	12 <animateMotion

	13 dur="4s"

	14 path="M100,100c0,200 200,200 200,0"

	15 repeatDur="12s"

	16 accumulate="sum"

	17 fill="freeze" />

	18 </circle>

	19 <circle cx="0" cy="0" r="60" stroke="#00f"

	20 stroke-width="21"

	21 fill="#f00">

	22 <animateMotion

	23 dur="4s"

	24 path="M100,100c0,200 200,200 200,0"

	25 repeatDur="12s"

	26 accumulate="sum"

	27 rotate="auto"

	28 fill="freeze" />

	29 </circle>

	30 <circle cx="0" cy="0" r="40"

	31 stroke="#00f" stroke-width="21"

	32 fill="#f00">

	33 <animateMotion

	34 dur="4s"

	35 path="M100,100c0,200 200,200 200,0"

	36 repeatDur="12s"

	37 accumulate="sum"

	38 rotate="auto-reverse"

	39 fill="freeze" />

	40 </circle>

	41 <circle cx="0" cy="0" r="20"

	42 stroke="#00f" stroke-width="21"

	43 fill="#f00">

	44 <animateMotion

	45 dur="4s"
	pdr. 2013/07/17 04:11:30 Minor nit: the indentation is wrong here Minor nit: the indentation is wrong here
	46 path="M100,100c0,200 200,200 200,0"

	47 repeatDur="12s"

	48 accumulate="sum"

	49 rotate="60"

	50 fill="freeze" />

	51 </circle>

	52 </g>

	53

	54 <g fill="none" stroke-width="20"

	55 stroke="#888" stroke-linecap="round" stroke-linejoin="round">

	56 <path d="M100,100c0,200 200,200 200,0">

	57 <set attributeName="display" to="none" begin="4s" />

	58 </path>
	pdr. 2013/07/17 04:11:30 Minor nit: the indentation is wrong here Minor nit: the indentation is wrong here
	59 <path d="M400,200c0,200 200,200 200,0">

	60 <set attributeName="display" to="none" begin="8s" />

	61 </path>

	62 <path d="M700,300c0,200 200,200 200,0" />

	63 </g>

	64 </g>

	65 </g>

	66 <script>

	67 if (window.testRunner)

	68 testRunner.waitUntilDone();

	69

	70 function loaded() {

	71 document.documentElement.setCurrentTime(12);
	pdr. 2013/07/17 04:11:30 Just a question: In this test and the others, the Just a question: In this test and the others, the setCurrentTime parameter is exactly equal to the repeatDur value of the animateMotion. Is there a reason for that? Do the tests still pass if you use 24 instead of 12? pavane 2013/07/17 05:30:17 anything above repeatDur value is fine as i am fre Show quoted text On 2013/07/17 04:11:30, pdr wrote: > Just a question: In this test and the others, the setCurrentTime parameter is > exactly equal to the repeatDur value of the animateMotion. Is there a reason for > that? Do the tests still pass if you use 24 instead of 12? anything above repeatDur value is fine as i am freezing the animation at the end.
	72

	73 if (window.testRunner)

	74 testRunner.notifyDone();

	75 }

	76 </script>

	77 </svg>

OLD	NEW