Bug 310045 – [macOS arm64] inspector/animation/lifecycle-css-transition.html is a flaky text failure/timeout

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Diego De La Toba]

inspector/animation/lifecycle-css-transition.html is a flaky text failure on release and timeout on debug

HISTORY:
https://results.webkit.org/?suite=layout-tests&test=inspector%2Fanimation%2Flifecycle-css-transition.html


TEXT DIFF:
--- /Volumes/Data/worker/Apple-Tahoe-Release-AppleSilicon-WK2-Tests/build/layout-test-results/inspector/animation/lifecycle-css-transition-expected.txt
+++ /Volumes/Data/worker/Apple-Tahoe-Release-AppleSilicon-WK2-Tests/build/layout-test-results/inspector/animation/lifecycle-css-transition-actual.txt
@@ -28,6 +28,5 @@
 
 Destroying animations...
 
-PASS: Animation destroyed.
-PASS: Removed animation has expected ID.
+!! TIMEOUT: took longer than 10000ms
 

DIFF URL:
https://build.webkit.org/results/Apple-Tahoe-Release-AppleSilicon-WK2-Tests/309331%40main%20(2464)/inspector/animation/lifecycle-css-transition-pretty-diff.html


HISTORY:
I was able to reproduce this on macOS release and debug with the following command:

run-webkit-tests --root <PATH-TO-SPADE> --<STYLE> --force --no-retry --clobber-old-results --iterations 1000 inspector/animation/lifecycle-css-transition.html

I am going to mark expectations as pass fail while this pends investigation.

[Radar WebKit Bug Importer]

<rdar://problem/172689579>

[Diego De La Toba]

Test gardening pull request: https://github.com/WebKit/WebKit/pull/60716

[EWS]

Test gardening commit 309345@main (d79772deea9c): <https://commits.webkit.org/309345@main>

Reviewed commits have been landed. Closing PR #60716 and removing active labels.

[Diego De La Toba]

This test awaits an Animation.animationDestroyed event after calling classList.remove("active"), but it appears the actual CSS transition cancellation requires a rendering update. Since destroyAnimations() uses setTimeout() to schedule GCController.collect(), no rendering update is guaranteed, and the test can time out waiting for the destruction event. Replacing setTimeout with requestAnimationFrame() forces a rendering update, ensuring the style change is processed and the CSS transition is cleaned up.

[Diego De La Toba]

Pull request: https://github.com/WebKit/WebKit/pull/60735

[EWS]

Committed 309384@main (b82442379b0d): <https://commits.webkit.org/309384@main>

Reviewed commits have been landed. Closing PR #60735 and removing active labels.