RESOLVED FIXED 52064
[chromium] Layout test fast/events/overflow-events.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=52064
Summary [chromium] Layout test fast/events/overflow-events.html is flaky
Stephen White
Reported 2011-01-07 08:39:58 PST
Seems to be only flaky on chromium-win debug, for now.
Attachments
Proposed unflakynessing try. (4.55 KB, patch)
2012-03-01 19:40 PST, Julien Chaffraix
no flags
Better unflakynessing. (4.45 KB, patch)
2012-03-02 09:00 PST, Julien Chaffraix
no flags
Peter Kasting
Comment 1 2011-08-25 09:53:55 PDT
Now also flaky on Mac and Linux debug.
Julien Chaffraix
Comment 2 2012-03-01 19:40:46 PST
Created attachment 129795 [details] Proposed unflakynessing try.
Stephen White
Comment 3 2012-03-02 04:29:05 PST
Comment on attachment 129795 [details] Proposed unflakynessing try. View in context: https://bugs.webkit.org/attachment.cgi?id=129795&action=review > LayoutTests/fast/events/overflow-events.html:38 > + if (currentTest == testResults.length - 1) { > + finished("SUCCESS"); > + return; > + } Since this comes before verification of the test results, won't it skip verification of the last test?
Julien Chaffraix
Comment 4 2012-03-02 08:49:58 PST
Comment on attachment 129795 [details] Proposed unflakynessing try. View in context: https://bugs.webkit.org/attachment.cgi?id=129795&action=review >> LayoutTests/fast/events/overflow-events.html:38 >> + } > > Since this comes before verification of the test results, won't it skip verification of the last test? I think you are right, let me tweak that so it's less error-prone.
Julien Chaffraix
Comment 5 2012-03-02 09:00:26 PST
Created attachment 129908 [details] Better unflakynessing.
WebKit Review Bot
Comment 6 2012-03-02 12:01:28 PST
Comment on attachment 129908 [details] Better unflakynessing. Clearing flags on attachment: 129908 Committed r109595: <http://trac.webkit.org/changeset/109595>
WebKit Review Bot
Comment 7 2012-03-02 12:01:33 PST
All reviewed patches have been landed. Closing bug.
Julien Chaffraix
Comment 8 2012-06-13 10:56:32 PDT
The test is not flaky anymore so I removed the entry in http://trac.webkit.org/changeset/120225
Note You need to log in before you can comment on or make changes to this bug.