Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1424)

Unified Diff: third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath.html

Issue 2227463002: SVG: Use path attribute for animateMotion (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: test Created 4 years, 4 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
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath-expected.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath.html
diff --git a/third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath.html b/third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath.html
new file mode 100644
index 0000000000000000000000000000000000000000..1a495998c128eb82d22ac7617c2678df5e8a5c96
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath.html
@@ -0,0 +1,30 @@
+<!DOCTYPE html>
+<style>
+#track {
+ d: path('M20,100 H180');
+ stroke: blue;
+}
+</style>
+<svg width="200" height="200" xmlns="http://www.w3.org/2000/svg"
fs 2016/08/08 15:45:38 Nit: Don't need to declare NS's (the HTML parser o
Eric Willigers 2016/08/10 01:13:15 Done.
+ xmlns:xlink="http://www.w3.org/1999/xlink">
+
+ <path id="track" d="M20,20 H180"/>
+
+ <circle r="10" cx="0" cy="0" fill="green">
+ <animateMotion dur="100ms" fill="freeze">
+ <mpath xlink:href="#track"/>
+ </animateMotion>
+ </circle>
+</svg>
+<script>
+'use strict';
+if (window.testRunner)
+ testRunner.waitUntilDone();
+
+requestAnimationFrame(function() {
fs 2016/08/08 15:45:38 If we can do the thing below, then I think we coul
Eric Willigers 2016/08/10 01:13:15 Done.
+ var track = document.getElementById('track');
+ track.setAttribute('d', 'M100,20 V180');
+ if (window.testRunner)
+ setTimeout(function(){testRunner.notifyDone();}, 100);
fs 2016/08/08 15:45:38 Could we avoid the 100ms delay by pausing the time
Eric Willigers 2016/08/10 01:13:15 The animation has fill="freeze", so now I simply u
+});
+</script>
« no previous file with comments | « no previous file | third_party/WebKit/LayoutTests/svg/animations/animateMotion_changingPath-expected.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698