Several animations/ tests flakily fail due to fragile setTimeout()-based framework 
https://bugs.webkit.org/show_bug.cgi?id=66953
Summary Several animations/ tests flakily fail due to fragile setTimeout()-based fram...
Peter Kasting
Reported 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?
Attachments
Add attachment proposed patch, testcase, etc.
Peter Kasting
Comment 1 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.
Peter Kasting
Comment 2 2011-11-16 16:01:56 PST
*** Bug 70988 has been marked as a duplicate of this bug. ***
Peter Kasting
Comment 3 2011-11-16 16:02:02 PST
*** Bug 71215 has been marked as a duplicate of this bug. ***
Peter Kasting
Comment 4 2011-11-16 16:02:08 PST
*** Bug 72485 has been marked as a duplicate of this bug. ***
Adam Barth
Comment 5 2011-11-16 17:00:01 PST
Yeah, the animation tests flaky out all the time. Fixing this systematically would be very beneficial.
Steve Block
Comment 6 2011-11-17 05:39:28 PST
Committed r100611: <http://trac.webkit.org/changeset/100611>
Steve Block
Comment 7 2011-11-17 08:26:33 PST
Committed r100626: <http://trac.webkit.org/changeset/100626>
Dean Jackson
Comment 8 2011-11-17 19:54:30 PST
I'm so sick of the current framework for testing. Sick and ashamed.
Note You need to log in before you can comment on or make changes to this bug.