Summary: | [macOS WK2 Debug ] Flaky Test: animations/play-state-in-shorthand.html | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Matt Lewis <jlewis3> | ||||
Component: | Animations | Assignee: | Antoine Quint <graouts> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, dino, graouts, ryanhaddad, simon.fraser, sroberts, tsavell, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=192503 | ||||||
Attachments: |
|
Description
Matt Lewis
2018-12-07 10:19:55 PST
Test still flaky on Mac WK2 Debug animations/play-state-in-shorthand.html Flakiness dashboard: https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=animations%2Fplay-state-in-shorthand.html Reproduce with: run-webkit-tests --root d240655 animations/play-state-in-shorthand.html --iterations 500 -f --debug Test was added in https://trac.webkit.org/changeset/200043/webkit Is flaky on most Mac builds, and recently started showing up on iOS Sim Release and Debug as well. Updated test expectations for Mac Debug and iOS Simulator in https://trac.webkit.org/changeset/244780/webkit . If I understand this test, it does the following (Simon, please correct me if I'm wrong): - set a timeout for 1s in the start callback to pause the animation using the CSS `animation` property shorthand halfway through - set various timeouts for the animated value to be tested at 500ms (before the start callback timeout has elapsed and the animation is paused), at 1000ms and 2500ms to check the animation has been paused. So the design of this test relies exclusively on setTimeout, which is bound to be flaky, especially as both the start callback and the second value check use the same timeout. Let's see if we can respect the design of this test while making is robust. I think it would be best to rewrite this test for the new animation engine to check the play state of the animation rather than be looking at CSS values. I'm going to do that, and leave the legacy variety of this test alone. Created attachment 380328 [details]
Patch
Comment on attachment 380328 [details] Patch Clearing flags on attachment: 380328 Committed r250781: <https://trac.webkit.org/changeset/250781> All reviewed patches have been landed. Closing bug. |