Chromium Code Reviews Chromium Code Reviews
Issue 201673003:
[SVG2] Add onbegin, onend and onrepeat EventHandlers on SVGAnimationElement (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/svg/animation-events.html |
| diff --git a/LayoutTests/fast/svg/animation-events.html b/LayoutTests/fast/svg/animation-events.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..82e9c31a920ddf9bcf92e9e3b5cb0fd72991e20a |
| --- /dev/null |
| +++ b/LayoutTests/fast/svg/animation-events.html |
| @@ -0,0 +1,26 @@ |
| +<!DOCTYPE html> |
| +<title>GlobalEventHandlers test</title> |
|
fs
2014/03/17 14:35:27
Nit: "Animation events test" (or something - not "
Erik Dahlström (inactive)
2014/03/17 15:52:37
Done.
|
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<script src="script-tests/animation-events.js"></script> |
| +<script> |
| +[ |
| + "onbegin", |
| + "onend", |
| + "onrepeat" |
| +].forEach(function(attribute) { |
| + testSet("SVGAnimateElement", attribute); |
|
fs
2014/03/17 14:35:27
Nested forEach? =) (I think that would help readab
Erik Dahlström (inactive)
2014/03/17 15:52:37
Somewhat changed now due to the added eventname te
|
| + testEnumerate("SVGAnimateElement", attribute); |
| + testReflect("SVGAnimateElement", attribute); |
| + testSet("SVGAnimateMotionElement", attribute); |
| + testEnumerate("SVGAnimateMotionElement", attribute); |
| + testReflect("SVGAnimateMotionElement", attribute); |
| + testSet("SVGAnimateTransformElement", attribute); |
| + testEnumerate("SVGAnimateTransformElement", attribute); |
| + testReflect("SVGAnimateTransformElement", attribute); |
| + testSet("SVGSetElement", attribute); |
| + testEnumerate("SVGSetElement", attribute); |
| + testReflect("SVGSetElement", attribute); |
| +}); |
| +</script> |
|
fs
2014/03/17 14:35:27
Any way to test that the mapping works?
Erik Dahlström (inactive)
2014/03/17 15:52:37
The event -> attribute name? Sure, I can add that.
|
| +<div id="log"></div> |
