Bug 66953

Summary: Several animations/ tests flakily fail due to fragile setTimeout()-based framework
Product: WebKit Reporter: Peter Kasting <pkasting>
Component: Tools / TestsAssignee: James Robinson <jamesr>
Status: NEW ---    
Severity: Normal CC: abarth, caseq, dino, jchaffraix, simon.fraser, steveblock
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   

Description Peter Kasting 2011-08-25 10:10:36 PDT
The following layout test is flaky on Chromium Snowleopard Debug (see http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=animations%2Fnegative-delay.html&group=%40DEPS%20-%20chromium.org ):

animations/negative-delay.html

No idea who should own this.  Dimitri, can you triage?
Comment 1 Peter Kasting 2011-11-16 16:00:47 PST
Morphing.  There are a number of animations tests with flaky text diffs all due to using what I consider a fragile, timer-based framework.  These are at least:

animations/animation-direction-normal.html
animations/change-keyframes.html
animations/change-one-anim.html
animations/combo-transform-translate+scale.html
animations/negative-delay.html
animations/play-state.html
animations/play-state-paused.html
animations/play-state-suspend.html
animations/suspend-resume-animation.html

There are other failing animations tests also, but I'm not sure whether they have the same causes.

We need to deflake these by making the animations tests use some more reliable design.

Ojan suggests James might be able to come up with an idea for a fix.
Comment 2 Peter Kasting 2011-11-16 16:01:56 PST
*** Bug 70988 has been marked as a duplicate of this bug. ***
Comment 3 Peter Kasting 2011-11-16 16:02:02 PST
*** Bug 71215 has been marked as a duplicate of this bug. ***
Comment 4 Peter Kasting 2011-11-16 16:02:08 PST
*** Bug 72485 has been marked as a duplicate of this bug. ***
Comment 5 Adam Barth 2011-11-16 17:00:01 PST
Yeah, the animation tests flaky out all the time.  Fixing this systematically would be very beneficial.
Comment 6 Steve Block 2011-11-17 05:39:28 PST
Committed r100611: <http://trac.webkit.org/changeset/100611>
Comment 7 Steve Block 2011-11-17 08:26:33 PST
Committed r100626: <http://trac.webkit.org/changeset/100626>
Comment 8 Dean Jackson 2011-11-17 19:54:30 PST
I'm so sick of the current framework for testing. Sick and ashamed.