WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
31646
LayoutTests/transitions/zero-duration-with-non-zero-delay-end.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=31646
Summary
LayoutTests/transitions/zero-duration-with-non-zero-delay-end.html is flaky
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+
Details
Formatted Diff
Diff
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.
Top of Page
Format For Printing
XML
Clone This Bug