LayoutTests/svg/animations/resources/animation-policy.svg - Issue 802143002: AnimationPolicy setting is applied to SVG animation.

Unified Diff: LayoutTests/svg/animations/resources/animation-policy.svg

Issue 802143002: AnimationPolicy setting is applied to SVG animation. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Update code Created 6 years 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

« no previous file with comments | « no previous file | LayoutTests/svg/animations/script-tests/svg-animation-policy.js » ('j') | LayoutTests/svg/animations/script-tests/svg-animation-policy.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: LayoutTests/svg/animations/resources/animation-policy.svg

diff --git a/LayoutTests/svg/animations/resources/animation-policy.svg b/LayoutTests/svg/animations/resources/animation-policy.svg

new file mode 100644

index 0000000000000000000000000000000000000000..25a3ace5658ff9589e3b487f8ef7a6423ca3821b

--- /dev/null

+++ b/LayoutTests/svg/animations/resources/animation-policy.svg

@@ -0,0 +1,14 @@

+<?xml version="1.0" encoding="UTF-8"?>

+<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 1.1 Tiny//EN" "http://www.w3.org/Graphics/SVG/1.1/DTD/svg11-tiny.dtd">

fs 2014/12/25 17:32:07 You can drop this.

je_julie(Not used) 2014/12/27 07:08:53 Done.

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

fs 2014/12/25 17:32:07 You don't need to declare the xlink namespace.

je_julie(Not used) 2014/12/27 07:08:53 Done.

+<animate id="an1" attributeName="visibility" to="visible" begin="0s" dur="3s" repeatCount="indefinite"/>

fs 2014/12/25 17:32:07 What case is this for? (AFAICS this will animate f

je_julie(Not used) 2014/12/27 07:08:53 I changed it with y, values="30;50;100" and discre

+<rect x="50" y="0" width="100" height="100" fill="green">

+ <set id="top" attributeName="y" to="30"

+ begin="0s; bottom.end" dur="1s" fill="freeze"/>

fs 2014/12/25 17:32:07 I don't see a need for the 'freeze's here. Could a

je_julie(Not used) 2014/12/27 07:08:53 Removed freeze. I removed set tags and I changed a

fs 2014/12/27 15:36:34 No, this should be fine.

+ <set id="middle" attributeName="y" to="50"

+ begin="top.end" dur="1s" fill="freeze"/>

+ <set id="bottom" attributeName="y" to="100"

fs 2014/12/25 17:32:07 Maybe align this with the other two <set>.

je_julie(Not used) 2014/12/27 07:08:53 Removed set tags.

+ begin="middle.end" dur="1s" fill="freeze"/>

+</rect>

+</svg>