Bug 31646

Summary: LayoutTests/transitions/zero-duration-with-non-zero-delay-end.html is flaky
Product: WebKit Reporter: Julie Parent <jparent>
Component: Tools / TestsAssignee: Julie Parent <jparent>
Status: RESOLVED FIXED    
Severity: Normal CC: cmarrin, dimich, eric
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows XP   
Attachments:
Description Flags
Increase timeout from 500ms to 750ms (transition delay is set to 500ms). eric: review+

Julie Parent
Reported 2009-11-18 14:04:01 PST
This test uses setTimeouts to measure the number of active transitions at given times. Since this is a 0 duration transition, it is not possible to use transitionEnd events instead (they are not dispatched), but it should be possible to improve reliability of this test by increasing the final timeout. This test fails ~50% of the time on the Chromium bots.
Attachments
Increase timeout from 500ms to 750ms (transition delay is set to 500ms). (1.12 KB, patch)
2009-11-18 14:14 PST, Julie Parent 		eric: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Julie Parent
Comment 1 2009-11-18 14:14:35 PST
Created attachment 43459 [details] Increase timeout from 500ms to 750ms (transition delay is set to 500ms).
Eric Seidel (no email)
Comment 2 2009-11-18 15:31:07 PST
Comment on attachment 43459 [details] Increase timeout from 500ms to 750ms (transition delay is set to 500ms). There is no event we could listen for instead? Ideally we would add a comment next to the setTimeout about why it's required. Especially since we're trying to kill all the other setTimeouts, seems we should document why this one is OK.
Eric Seidel (no email)
Comment 3 2009-11-18 15:31:30 PST
Also, you probably want to cc Chris on transition bugs. :)
Julie Parent
Comment 4 2009-11-18 15:33:01 PST
Since the transitionEnd event isn't dispatched, I'm not sure what other event we could use. I'll add a comment with the setTimeout. What is Chris's email address?
Eric Seidel (no email)
Comment 5 2009-11-18 15:42:38 PST
cmarrin. I CC'd him already. Thanks.
Julie Parent
Comment 6 2009-11-18 15:58:47 PST
Committed: http://trac.webkit.org/changeset/51150
Note You need to log in before you can comment on or make changes to this bug.