commit 74467d3df710327c1d28b6bf73679bd4aaedbdc3 parent 2828fff2c97520e52522922a2e7eb4237d6d6f41 Author: David Awogbemila <awogbemila@chromium.org> Date: Wed, 15 Oct 2025 08:43:29 +0000 Bug 1993221 [wpt PR 55285] - [animation-trigger] Update animation-trigger parsing, a=testonly Automatic update from web-platform-tests [animation-trigger] Update animation-trigger parsing The CSSWG resolved to remove "trigger()" from the animation-trigger syntax and to specify events at the source and not the target. The syntax is now the following: animation-trigger: [ [<dashed-ident> <enter-behavior> <exit-behavior>?]+ ]# [1] https://github.com/w3c/csswg-drafts/issues/12652#issuecomment-3329841437 Bug: 390314945 Change-Id: I2724b498dfe686aa85537470d636957c09411f42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/7004885 Commit-Queue: David Awogbemila <awogbemila@chromium.org> Reviewed-by: Anders Hartvoll Ruud <andruud@chromium.org> Reviewed-by: Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/main@{#1528227} -- wpt-commits: 9b489c0284dd15d4ecac133f96e0f6ecc63604f1 wpt-pr: 55285 Diffstat:
19 files changed, 78 insertions(+), 144 deletions(-)
diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-addAnimation.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-addAnimation.tentative.html @@ -78,8 +78,7 @@ // As the default trigger is always in the tripped state, the animation // should be played right away. - trigger.addAnimation(animation, "enter", "play-forwards"); - trigger.addAnimation(animation, "exit", "play-backwards"); + trigger.addAnimation(animation, "play-forwards", "play-backwards"); await animation.finished; assert_times_equal(animation.currentTime, ANIMATION_DURATION_MS, @@ -98,8 +97,7 @@ assert_equals(scroller.scrollTop, 0, "scroller is not scrolled, i.e. " + "not within the trigger range"); - trigger.addAnimation(animation, "enter", "play-forwards"); - trigger.addAnimation(animation, "exit", "play-backwards"); + trigger.addAnimation(animation, "play-forwards", "play-backwards"); await waitForAnimationFrames(2); @@ -135,8 +133,7 @@ assert_equals(animation.playState, "idle", "animation is still idle"); assert_equals(animation.currentTime, null, "currentTime is still null"); - trigger.addAnimation(animation, "enter", "play-forwards"); - trigger.addAnimation(animation, "exit", "play-backwards"); + trigger.addAnimation(animation, "play-forwards", "play-backwards"); await animation.finished; assert_equals(animation.playState, "finished", diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-alternate.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-alternate.tentative.html @@ -24,18 +24,15 @@ } #scroll_target { - animation-trigger: - trigger(--scrolltrigger, enter play-forwards, exit play-backwards); + animation-trigger: --scrolltrigger play-forwards play-backwards; timeline-trigger: --scrolltrigger scroll() 150px 200px; } #view_target { - animation-trigger: - trigger(--viewtrigger, enter play-forwards, exit play-backwards); + animation-trigger: --viewtrigger play-forwards play-backwards; timeline-trigger: --viewtrigger view() 150px 200px; } #deferred_target { - animation-trigger: - trigger(--deferredtrigger, enter play-forwards, exit play-backwards); + animation-trigger: --deferredtrigger play-forwards play-backwards; } #deferred_subject { view-timeline: --viewtimeline; diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-disarmed-by-apis.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-disarmed-by-apis.tentative.html @@ -109,8 +109,7 @@ assert_equals(animation.playState, "idle", "animation is idle."); // Attaching the trigger should simply keep the animaiton paused at // zero currentTime. - trigger.addAnimation(animation, "enter", "play-forwards"); - trigger.addAnimation(animation, "exit", "play-backwards"); + trigger.addAnimation(animation, "play-forwards", "play-backwards"); await waitForNextFrame(); assert_times_equal(animation.currentTime, 0, "animation is paused at currentTime zero."); diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-fill-mode-both.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-fill-mode-both.tentative.html @@ -25,7 +25,7 @@ background-color: blue; animation: slide-in 0.3s both; timeline-trigger: --trigger view() contain 0% contain 100%; - animation-trigger: trigger(--trigger, enter play-forwards, exit play-backwards); + animation-trigger: --trigger play-forwards play-backwards; position: absolute; } .space { diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-fill-mode-none.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-fill-mode-none.tentative.html @@ -24,7 +24,7 @@ background-color: blue; animation: slide-in 3s none; timeline-trigger: --trigger view() contain 0% contain 100%; - animation-trigger: trigger(--trigger, enter play-forwards, exit play-backwards); + animation-trigger: --trigger play-forwards play-backwards; } .space { height: 250px; diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-late-attached-timeline.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-late-attached-timeline.tentative.html @@ -77,7 +77,7 @@ // Now attach a view timeline. Since we are in the trigger range, we // should trigger. await testAnimationTrigger(test, async () => { - target.style.animationTrigger = "trigger(--trigger, enter play-once)"; + target.style.animationTrigger = "--trigger play-once"; target.style.timelineTriggerSource = "--viewtimeline"; await waitForAnimationFrames(5); }, target, ["animationstart", "animationend", "animationcancel"], diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-multiple-animations.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-multiple-animations.tentative.html @@ -91,8 +91,7 @@ assert_equals(scroller.scrollTop, 0, "scroller is not scrolled, i.e. not within the trigger range"); - trigger.addAnimation(animation1, "enter", "play-forwards"); - trigger.addAnimation(animation1, "exit", "play-backwards"); + trigger.addAnimation(animation1, "play-forwards", "play-backwards"); assert_equals(animation1.playState, "paused", "animation1 is paused, awaiting trigger event"); @@ -102,8 +101,7 @@ assert_equals(animation2.currentTime, null, "animations2's currentTime is null"); - trigger.addAnimation(animation2, "enter", "play-forwards"); - trigger.addAnimation(animation2, "exit", "play-backwards"); + trigger.addAnimation(animation2, "play-forwards", "play-backwards"); assert_equals(animation1.playState, "paused", "animation is paused, awaiting trigger event"); diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-multiple-triggers.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-multiple-triggers.tentative.html @@ -96,10 +96,8 @@ assert_equals(scroller.scrollTop, 0, "scroller is not scrolled, i.e. " + "not within the trigger range"); - trigger1.addAnimation(animation, "enter", "play-forwards"); - trigger1.addAnimation(animation, "exit", "reset"); - trigger2.addAnimation(animation, "enter", "play-forwards"); - trigger2.addAnimation(animation, "exit", "reset"); + trigger1.addAnimation(animation, "play-forwards", "reset"); + trigger2.addAnimation(animation, "play-forwards", "reset"); // Test preconditions. assert_equals(animation.playState, "paused", "animation is idle"); @@ -146,10 +144,8 @@ assert_equals(scroller.scrollTop, 0, "scroller is not scrolled, i.e. " + "not within the trigger range"); - trigger1.addAnimation(animation, "enter", "play-forwards"); - trigger1.addAnimation(animation, "exit", "reset"); - trigger2.addAnimation(animation, "enter", "play-forwards"); - trigger2.addAnimation(animation, "exit", "reset"); + trigger1.addAnimation(animation, "play-forwards", "reset"); + trigger2.addAnimation(animation, "play-forwards", "reset"); // Test preconditions. assert_equals(animation.playState, "paused", "animation is paused"); diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-once-play-state.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-once-play-state.tentative.html @@ -25,7 +25,7 @@ .target { animation: myAnim linear 0.5s forwards; timeline-trigger: --trigger --viewtimeline 150px 200px; - animation-trigger: trigger(--trigger, enter play-once); + animation-trigger: --trigger play-once; } .scroller { overflow-y: scroll; diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-once.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-once.tentative.html @@ -24,15 +24,15 @@ } #scroll_target { - animation-trigger: trigger(--scrolltrigger, enter play-once); + animation-trigger: --scrolltrigger play-once; timeline-trigger: --scrolltrigger scroll() 150px 200px; } #view_target { - animation-trigger: trigger(--viewtrigger, enter play-once); + animation-trigger: --viewtrigger play-once; timeline-trigger: --viewtrigger view() 150px 200px; } #deferred_target { - animation-trigger: trigger(--deferredtrigger, enter play-once); + animation-trigger: --deferredtrigger play-once; } #deferred_subject { view-timeline: --viewtimeline; diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-repeat.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-repeat.tentative.html @@ -24,15 +24,15 @@ } #scroll_target { - animation-trigger: trigger(--scrolltrigger, enter play-forwards, exit reset); + animation-trigger: --scrolltrigger play-forwards reset; timeline-trigger: --scrolltrigger scroll() 250px 300px 200px 350px; } #view_target { - animation-trigger: trigger(--viewtrigger, enter play-forwards, exit reset); + animation-trigger: --viewtrigger play-forwards reset; timeline-trigger: --viewtrigger view() 250px 300px 200px 350px; } #deferred_target { - animation-trigger: trigger(--deferredtrigger, enter play-forwards, exit reset); + animation-trigger: --deferredtrigger play-forwards reset; } #deferred_subject { view-timeline: --viewtimeline; diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-state.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/animation-trigger-state.tentative.html @@ -24,15 +24,15 @@ } #scroll_target { - animation-trigger: trigger(--scrolltrigger, enter play-once, exit pause); + animation-trigger: --scrolltrigger play-once pause; timeline-trigger: --scrolltrigger scroll() 150px 200px; } #view_target { - animation-trigger: trigger(--viewtrigger, enter play-once, exit pause); + animation-trigger: --viewtrigger play-once pause; timeline-trigger: --viewtrigger view() 150px 200px; } #deferred_target { - animation-trigger: trigger(--deferredtrigger, enter play-once, exit pause); + animation-trigger: --deferredtrigger play-once pause; } #deferred_subject { view-timeline: --viewtimeline; diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/parsing/animation-trigger-parsing.tentative.html b/testing/web-platform/tests/scroll-animations/animation-trigger/parsing/animation-trigger-parsing.tentative.html @@ -13,126 +13,65 @@ test_valid_value('animation-trigger', 'revert'); // Valid: Single attachment per animation. test_valid_value('animation-trigger', 'none'); -test_valid_value('animation-trigger', 'trigger(--abc, click play)'); -test_valid_value('animation-trigger', - ' trigger(--abc, click play)', 'trigger(--abc, click play)'); -test_valid_value('animation-trigger', 'trigger(--aBc, click play)'); -test_valid_value('animation-trigger', - 'trigger(--foo, click play), trigger(--bar, click play)'); -test_valid_value('animation-trigger', - 'trigger(--bar, click play), trigger(--foo, click play)'); +test_valid_value('animation-trigger', '--abc play'); +test_valid_value('animation-trigger', ' --abc play', '--abc play'); +test_valid_value('animation-trigger', '--aBc play'); +test_valid_value('animation-trigger', '--aBc play pause'); +test_valid_value('animation-trigger', '--foo play pause, --bar play pause'); +test_valid_value('animation-trigger', '--bar play reset, --foo play reset'); test_valid_value('animation-trigger', 'none, none'); -test_valid_value('animation-trigger', - 'trigger(--a, click play), none, trigger(--b, click play)'); -// Valid: Single attachment per animation, multiple action-behavior pairs -test_valid_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause)'); -test_valid_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause, touch reset)'); -test_valid_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause, touch reset), ' + - 'trigger(--def, click pause, dblclick reset, touch play)'); +test_valid_value('animation-trigger', '--a play pause, none, --b play reset'); // Valid: Multiple attachments per animation. +test_valid_value('animation-trigger', '--abc play --abc play-pause'); test_valid_value('animation-trigger', - 'trigger(--abc, click play) trigger(--abc, click play)'); + '--abc play --bcd play-pause, --cde play-alternate --def play reset'); test_valid_value('animation-trigger', - 'trigger(--abc, click play) trigger(--bcd, click play), ' + - 'trigger(--cde, click play) trigger(--def, click play)'); + ' --abc play --bcd reset --cde pause --def play-backwards', + '--abc play --bcd reset --cde pause --def play-backwards'); +test_valid_value('animation-trigger', '--aBc play --abc play --aBc play'); test_valid_value('animation-trigger', - ' trigger(--abc, click play) trigger(--bcd, click play), ' + - ' trigger(--cde, click play) trigger(--def, click play)', - 'trigger(--abc, click play) trigger(--bcd, click play), ' + - 'trigger(--cde, click play) trigger(--def, click play)'); -test_valid_value('animation-trigger', -'trigger(--aBc, click play) trigger(--abc, click play) ' + -'trigger(--aBc, click play)' -); -test_valid_value('animation-trigger', -'trigger(--aBc, click play) trigger(--abc, click play), ' + -'trigger(--aBc, click play)' -); -test_valid_value('animation-trigger', 'trigger(---aBc, click play)'); -test_valid_value('animation-trigger', 'trigger(--aBc-dEf, click play)'); -test_valid_value('animation-trigger', 'trigger(---aBc-dEf, click play)'); -// Valid: Multiple attachments per animation, multiple action-behavior pairs. -test_valid_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause) ' + - 'trigger(--def, click play, dblclick reset)'); + '--aBc play --abc play-forwards, --aBc play-backwards'); +test_valid_value('animation-trigger', '---aBc play'); +test_valid_value('animation-trigger', '--aBc-dEf play'); +test_valid_value('animation-trigger', '---aBc-dEf play'); // Computed: Single attachment per animation. test_computed_value('animation-trigger', 'none'); -test_computed_value('animation-trigger', 'trigger(--abc, click play)'); -test_computed_value('animation-trigger',' trigger(--abc, click play)', - 'trigger(--abc, click play)'); -test_computed_value('animation-trigger', 'trigger(--aBc, click play)'); -test_computed_value('animation-trigger', 'trigger(--foo, click play), ' + - 'trigger(--bar, click play)'); -test_computed_value('animation-trigger', 'trigger(--bar, click play), ' + - 'trigger(--foo, click play)'); +test_computed_value('animation-trigger', '--abc play'); +test_computed_value('animation-trigger',' --abc play', '--abc play'); +test_computed_value('animation-trigger', '--aBc play'); +test_computed_value('animation-trigger', '--foo play, --bar pause'); +test_computed_value('animation-trigger', '--bar play, --foo pause'); test_computed_value('animation-trigger', 'none, none'); -test_computed_value('animation-trigger', 'trigger(--aBc, click play), ' + - 'trigger(--aBc, click play)'); -// Computed: Single attachment per animation, multiple action-behavior pairs -test_computed_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause)'); -test_computed_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause, touch reset)'); -test_computed_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause, touch reset), ' + - 'trigger(--def, click pause, dblclick reset, touch play)'); -// Computed: Multiple attachments per animation, multiple action-behavior pairs. -test_computed_value('animation-trigger', - 'trigger(--abc, click play, dblclick pause) ' + - 'trigger(--def, click play, dblclick reset)'); +test_computed_value('animation-trigger', '--aBc play, --aBc reset'); // Computed: Multiple attachments per animation +test_computed_value('animation-trigger', '--abc play --abc play'); test_computed_value('animation-trigger', - 'trigger(--abc, click play) trigger(--abc, click play)'); -test_computed_value('animation-trigger', - 'trigger(--abc, click play) trigger(--bcd, click play), ' + - 'trigger(--abc, click play) trigger(--bcd, click play)'); -test_computed_value('animation-trigger', - 'trigger(--abc, click play) trigger(--bcd, click play), ' + - 'trigger(--cde, click play) trigger(--def, click play)'); -test_computed_value('animation-trigger', - ' trigger(--abc, click play) trigger(--abc, click play)', - 'trigger(--abc, click play) trigger(--abc, click play)'); + '--abc play --bcd pause, --abc play --bcd play'); test_computed_value('animation-trigger', - ' trigger(--abc, click play) trigger(--bcd, click play)', - 'trigger(--abc, click play) trigger(--bcd, click play)'); + '--abc play --bcd play, --cde play --def play'); test_computed_value('animation-trigger', - 'trigger(--aBc, click play) trigger(--abc, click play)'); + ' --abc play --abc play', '--abc play --abc play'); test_computed_value('animation-trigger', - 'trigger(--aBc, click play) trigger(--abc, click play) trigger(--aBc, click play)'); -test_computed_value('animation-trigger', - 'trigger(--a, click play), none, trigger(--b, click play)'); -test_computed_value('animation-trigger', 'trigger(---abc, click play)'); + ' --abc play --bcd play', '--abc play --bcd play'); +test_computed_value('animation-trigger', '--aBc play --abc play'); +test_computed_value('animation-trigger', '--aBc play --abc play --aBc play'); +test_computed_value('animation-trigger', '--a play, none, --b play'); +test_computed_value('animation-trigger', '---abc play'); // Invalid -test_invalid_value('animation-trigger', 'trigger(--abc, click play) none'); -test_invalid_value('animation-trigger', - 'trigger(--a, click play) trigger(--b, click play), trigger(--a, click play) none, trigger(--b, click play)'); -test_invalid_value('animation-trigger', 'none trigger(--abc, click play)'); test_invalid_value('animation-trigger', 'none none'); test_invalid_value('animation-trigger', 'none none, none', 'none, none'); test_invalid_value('animation-trigger', 'auto'); test_invalid_value('animation-trigger', 'abc'); test_invalid_value('animation-trigger', '--abc'); -test_invalid_value('animation-trigger', 'trigger'); -test_invalid_value('animation-trigger', 'trigger(--abc'); -test_invalid_value('animation-trigger', 'trigger()'); -test_invalid_value('animation-trigger', 'trigger(click)'); -test_invalid_value('animation-trigger', 'trigger(play)'); -test_invalid_value('animation-trigger', 'trigger(-abc, click play)'); -test_invalid_value('animation-trigger', 'trigger(--abc)'); -test_invalid_value('animation-trigger', 'trigger(--abc, click)'); -test_invalid_value('animation-trigger', 'trigger(--abc, play)'); -test_invalid_value('animation-trigger', 'trigger(--abc click play)'); -test_invalid_value('animation-trigger', 'trigger(--abc, click play click play)'); -test_invalid_value('animation-trigger', 'trigger(--abc click play, click play)'); -test_invalid_value('animation-trigger', 'trigger(--abc, click, play)'); -test_invalid_value('animation-trigger', 'trigger(--abc, click, click play)'); +test_invalid_value('animation-trigger', '-abc play'); +test_invalid_value('animation-trigger', '--abc '); +test_invalid_value('animation-trigger', 'play, play'); +test_invalid_value('animation-trigger', '--abc play, play'); +test_invalid_value('animation-trigger', '--abc, --def play'); test_invalid_value('animation-trigger', 'default'); test_invalid_value('animation-trigger', '10px'); test_invalid_value('animation-trigger', 'foo bar'); @@ -140,4 +79,14 @@ test_invalid_value('animation-trigger', '"foo" "bar"'); test_invalid_value('animation-trigger', 'rgb(1, 2, 3)'); test_invalid_value('animation-trigger', '#fefefe'); test_invalid_value('animation-trigger', '--a,, --b'); +test_invalid_value('animation-trigger', 'trigger(--abc, play) none'); +test_invalid_value('animation-trigger', 'none trigger(--abc, play)'); +test_invalid_value('animation-trigger', 'trigger'); +test_invalid_value('animation-trigger', 'trigger(--abc'); +test_invalid_value('animation-trigger', 'trigger()'); +test_invalid_value('animation-trigger', 'trigger(--abc play)'); +// Invalid: multiple behaviors. +test_invalid_value('animation-trigger', '--abc play pause reset'); +test_invalid_value('animation-trigger', + '--abc play pause reset, --def pause reset play'); </script> \ No newline at end of file diff --git a/testing/web-platform/tests/scroll-animations/animation-trigger/support/support.js b/testing/web-platform/tests/scroll-animations/animation-trigger/support/support.js @@ -105,14 +105,12 @@ function setupAnimationAndTrigger(target, subject, duration) { )); let trigger = new TimelineTrigger({ - behavior: "alternate", timeline: new ViewTimeline({ subject: subject, axis: "y" }), rangeStart: "contain 0%", rangeEnd: "contain 100%" }); - trigger.addAnimation(animation, "enter", "play-forwards"); - trigger.addAnimation(animation, "exit", "play-backwards"); + trigger.addAnimation(animation, "play-forwards", "play-backwards"); } async function waitForAnimation(targetCurrentTime, animation) { diff --git a/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-alternate.tentative.html b/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-alternate.tentative.html @@ -30,7 +30,7 @@ eventType: "click", eventTarget: eventTarget }); - trigger.addAnimation(animation, "click", "play-alternate"); + trigger.addAnimation(animation, "play-alternate"); await waitForAnimationFrames(1); assert_equals(animation.playState, "idle", "animation is idle"); assert_equals(animation.playbackRate, 1, "playbackRate is initially 1"); diff --git a/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-before-handlers.tentative.html b/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-before-handlers.tentative.html @@ -37,7 +37,7 @@ eventType: "click", eventTarget: eventTarget }); - trigger.addAnimation(animation, "click", "play"); + trigger.addAnimation(animation, "play"); await waitForAnimationFrames(2); assert_equals(animation.playState, "idle", "animation is idle"); await test_driver.click(eventTarget); diff --git a/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-once.tentative.html b/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-once.tentative.html @@ -30,7 +30,7 @@ eventType: "click", eventTarget: eventTarget }); - trigger.addAnimation(animation, "click", "play-once"); + trigger.addAnimation(animation, "play-once"); await waitForAnimationFrames(1); assert_equals(animation.playState, "idle", "animation is idle"); await test_driver.click(eventTarget); diff --git a/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-repeat.tentative.html b/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-repeat.tentative.html @@ -30,7 +30,7 @@ eventType: "click", eventTarget: eventTarget }); - trigger.addAnimation(animation, "click", "replay"); + trigger.addAnimation(animation, "replay"); await waitForAnimationFrames(1); assert_equals(animation.playState, "idle", "animation is idle"); await test_driver.click(eventTarget); diff --git a/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-state.tentative.html b/testing/web-platform/tests/web-animations/animation-trigger/event-trigger-state.tentative.html @@ -30,7 +30,7 @@ eventType: "click", eventTarget: eventTarget }); - trigger.addAnimation(animation, "click", "play-pause"); + trigger.addAnimation(animation, "play-pause"); await waitForAnimationFrames(1); assert_equals(animation.playState, "idle", "animation is idle"); await test_driver.click(eventTarget);