Bug 1975213 [wpt PR 53534] - Allow trailing semicolon in 'keyTimes' and 'keyPoints', a=testonly

Fredrik Söderquist · moz-wptsync-bot · commit e7a24b9b3f2b · 2025-07-12T09:15:41.000Z
Automatic update from web-platform-tests Allow trailing semicolon in 'keyTimes' and 'keyPoints' The spec [1][2] has allowed trailing semicolons for the these attributes for a time now, and we're allowing trailing semicolons in 'values' and 'keySplines' already. Guarded by "SmilKeyTimesKeyPointsAllowTrailingSemicolon". [1] https://svgwg.org/specs/animations/#KeyTimesAttribute [2] https://svgwg.org/specs/animations/#KeyPointsAttribute Fixed: 427020732 Change-Id: I29fb8bab7672b1f3be9352f53ff74a103a3febd2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6695946 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/main@{#1481457} -- wpt-commits: 526c6655fc7755f9025f3b1e073aaa000eb678c0 wpt-pr: 53534
diff --git a/testing/web-platform/tests/svg/animations/scripted/keypoints-attribute-trailing-semi.html b/testing/web-platform/tests/svg/animations/scripted/keypoints-attribute-trailing-semi.html
@@ -0,0 +1,20 @@
+<!doctype html>
+<title>A trailing semicolon is allowed in the 'keyPoints' attribute</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/common/rendering-utils.js"></script>
+<svg>
+  <rect width="100" height="100" x="100" fill="blue">
+    <animateMotion dur="10ms" fill="freeze"
+                   values="0,0; -100,0" keyTimes="0; 1" keyPoints="0; 1;"/>
+  </rect>
+</svg>
+<script>
+  promise_test(async t => {
+    const animation = document.querySelector('rect > animateMotion');
+    const watcher = new EventWatcher(t, animation, 'endEvent');
+    await watcher.wait_for('endEvent');
+    await waitForAtLeastOneFrame();
+    assert_equals(animation.targetElement.getCTM().e, -100);
+  });
+</script>
diff --git a/testing/web-platform/tests/svg/animations/scripted/keytimes-attribute-trailing-semi.html b/testing/web-platform/tests/svg/animations/scripted/keytimes-attribute-trailing-semi.html
@@ -0,0 +1,20 @@
+<!doctype html>
+<title>A trailing semicolon is allowed in the 'keyTimes' attribute</title>
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/common/rendering-utils.js"></script>
+<svg>
+  <rect width="100" height="100" fill="blue" opacity="0">
+    <animate attributeName="opacity" dur="10ms" fill="freeze"
+             values="0; 1" keyTimes="0; 1;"/>
+  </rect>
+</svg>
+<script>
+  promise_test(async t => {
+    const animation = document.querySelector('rect > animate');
+    const watcher = new EventWatcher(t, animation, 'endEvent');
+    await watcher.wait_for('endEvent');
+    await waitForAtLeastOneFrame();
+    assert_equals(getComputedStyle(animation.targetElement).opacity, "1");
+  });
+</script>
