Bug 52064 – [chromium] Layout test fast/events/overflow-events.html is flaky

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Stephen White]

Seems to be only flaky on chromium-win debug, for now.

[Peter Kasting]

Now also flaky on Mac and Linux debug.

[Julien Chaffraix]

Created attachment 129795 [details]
Proposed unflakynessing try.

[Stephen White]

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 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]

Created attachment 129908 [details]
Better unflakynessing.

[WebKit Review Bot]

Comment on attachment 129908 [details]
Better unflakynessing.

Clearing flags on attachment: 129908

Committed r109595: <http://trac.webkit.org/changeset/109595>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Julien Chaffraix]

The test is not flaky anymore so I removed the entry in http://trac.webkit.org/changeset/120225