Bug 196869

Summary: [win] REGRESSION (r244182): Animation layout test failures
Product: WebKit Reporter: Ryan Haddad <ryanhaddad>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: NEW ---    
Severity: Normal CC: ap, Hironori.Fujii, pvollan, sabouhallawa, webkit-bot-watchers-bugzilla, 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=177484

Description Ryan Haddad 2019-04-12 11:22:00 PDT
The following layout tests are failing on Windows bots after https://trac.webkit.org/changeset/244182/webkit:

animations/animation-multiple-callbacks-timestamp.html
legacy-animation-engine/animations/animation-multiple-callbacks-timestamp.html
imported/mozilla/css-animations/test_animation-currenttime.html

Results:
https://build.webkit.org/results/Apple%20Win%207%20Release%20(Tests)/r244216%20(2074)/results.html#

Flakiness dashboard:
https://webkit-test-results.webkit.org/dashboards/flakiness_dashboard.html#tests=animations%2Fanimation-multiple-callbacks-timestamp.html%20legacy-animation-engine%2Fanimations%2Fanimation-multiple-callbacks-timestamp.html%20imported%2Fmozilla%2Fcss-animations%2Ftest_animation-currenttime.html

--- /home/buildbot/worker/win7-release-tests/build/layout-test-results/animations/animation-multiple-callbacks-timestamp-expected.txt
+++ /home/buildbot/worker/win7-release-tests/build/layout-test-results/animations/animation-multiple-callbacks-timestamp-actual.txt
@@ -2,4 +2,5 @@
 PASS successfullyParsed is true
 
 TEST COMPLETE
+FAIL timestamp = 490, timestamp1 = 488, window.performance.now() = 490
 

--- /home/buildbot/worker/win7-release-tests/build/layout-test-results/imported/mozilla/css-animations/test_animation-currenttime-expected.txt
+++ /home/buildbot/worker/win7-release-tests/build/layout-test-results/imported/mozilla/css-animations/test_animation-currenttime-actual.txt
@@ -11,7 +11,7 @@
 PASS Seeking finished -> paused dispatches animationstart 
 PASS Setting currentTime to null 
 PASS Animation.currentTime after pausing 
-PASS Animation.currentTime clamping 
+FAIL Animation.currentTime clamping assert_equals: Animation.currentTime should not continue to increase after the animation has finished expected 100000 but got 100018
 PASS Animation.currentTime clamping for reversed animation 
 PASS Animation.currentTime after cancelling 
 PASS After aborting a pause when finished, the call to play() should rewind the current time 

https://build.webkit.org/results/Apple%20Win%207%20Release%20(Tests)/r244216%20(2074)/legacy-animation-engine/fast/layers/no-clipping-overflow-hidden-added-after-transform-diffs.html
Comment 1 Ryan Haddad 2019-04-12 11:26:28 PDT
(In reply to Ryan Haddad from comment #0)
> https://build.webkit.org/results/Apple%20Win%207%20Release%20(Tests)/
> r244216%20(2074)/legacy-animation-engine/fast/layers/no-clipping-overflow-
> hidden-added-after-transform-diffs.html

This test (legacy-animation-engine/fast/layers/no-clipping-overflow-hidden-added-after-transform.html) has actually be a flaky failure for a while. It did not regress with r244182.
Comment 2 Ryan Haddad 2019-04-12 11:35:32 PDT
Marked the following as failing:

animations/animation-multiple-callbacks-timestamp.html
legacy-animation-engine/animations/animation-multiple-callbacks-timestamp.html
imported/mozilla/css-animations/test_animation-currenttime.html

https://trac.webkit.org/changeset/244227/webkit
Comment 3 Radar WebKit Bug Importer 2019-04-12 22:38:04 PDT
<rdar://problem/49873249>