Chromium Code Reviews Chromium Code Reviews
Issue 2618323002:
Block animation of the SVGScriptElement (Closed)
| Index: third_party/WebKit/LayoutTests/svg/animations/script-href-no-animate.html |
| diff --git a/third_party/WebKit/LayoutTests/svg/animations/script-href-no-animate.html b/third_party/WebKit/LayoutTests/svg/animations/script-href-no-animate.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..4d7b8e6e1654726e7aea3693eb32e5ddd4d6ccd2 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/svg/animations/script-href-no-animate.html |
| @@ -0,0 +1,25 @@ |
| +<!DOCTYPE html> |
| +<title>SVGScriptElement.href should not be animatable</title> |
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<script> |
| +var foo = 42; |
| + |
| +async_test(t => { |
| + window.onload = () => { |
| + var s = document.createElementNS('http://www.w3.org/2000/svg', 'script'); |
| + s.onload = t.unreached_func('Should not get a load event'); |
| + s.id = 'x'; |
| + document.querySelector('svg').appendChild(s); |
| + requestAnimationFrame(_ => { |
| + requestAnimationFrame(t.step_func_done(_ => { |
|
Mike West
2017/01/09 19:31:19
Why two frames? I'll defer to y'all with regard to
fs
2017/01/10 10:15:16
This is a bit of a frailty with the test I'm afrai
|
| + assert_equals(foo, 42); |
| + })); |
| + }); |
| + }; |
| +}); |
| +</script> |
| +<svg> |
| + <rect width="100" height="100" fill="green"/> |
| + <set href="#x" attributeName="href" to="resources/set-foo.js"/> |
|
Mike West
2017/01/09 19:31:19
Would you mind adding a similar test for `<animate
fs
2017/01/10 10:15:16
I can do that.
|
| +</svg> |
