Bug 196869 - [win] REGRESSION (r244182): Animation layout test failures
Summary: [win] REGRESSION (r244182): Animation layout test failures
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-04-12 11:22 PDT by Ryan Haddad
Modified: 2019-04-25 19:03 PDT (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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>