247884 2022-11-14 02:31:21 -0800 [css-transitions] setting transition-property to "none" does not disassociate CSS Transition from owning element 2022-11-14 12:27:21 -0800 1 1 1 Unclassified WebKit Animations Safari Technology Preview Unspecified Unspecified RESOLVED FIXED InRadar, WebExposed, WPTImpact P2 Normal --- 237331 1 graouts graouts dino graouts koivisto webkit-bug-importer oldest_to_newest 1912248 0 graouts 2022-11-14 02:31:21 -0800 We are failing a subtest in web-animations/timing-model/timelines/update-and-send-events-replacement.html: promise_test(async t => { // Setup transition const div = createDiv(t); div.style.opacity = '0'; div.style.transition = 'opacity 1ms'; getComputedStyle(div).opacity; div.style.opacity = '1'; const cssTransition = div.getAnimations()[0]; cssTransition.effect.updateTiming({ fill: 'forwards' }); // Break tie to markup div.style.transitionProperty = 'none'; assert_equals(cssTransition.playState, 'idle'); // Restart transition cssTransition.play(); const scriptAnimation = div.animate( { opacity: 1 }, { duration: 1, fill: 'forwards', } ); await scriptAnimation.finished; assert_equals(cssTransition.replaceState, 'removed'); assert_equals(scriptAnimation.replaceState, 'active'); }, 'Removes a CSS transition no longer tied to markup'); We fail the second-to-last assertion because we do not correctly disassociate a CSS Transition from its owning element when transition-property is set to "none" and thus do not mark that transition as replaceable. 1912253 1 graouts 2022-11-14 02:43:47 -0800 Pull request: https://github.com/WebKit/WebKit/pull/6465 1912397 2 ews-feeder 2022-11-14 12:26:22 -0800 Committed 256666@main (7821448df0f4): <https://commits.webkit.org/256666@main> Reviewed commits have been landed. Closing PR #6465 and removing active labels. 1912398 3 webkit-bug-importer 2022-11-14 12:27:21 -0800 <rdar://problem/102333068>