Bug 71880 – [Chromium] Layout Test fast/forms/select-clientheight-large-size.html is timing out

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Yuta Kitamura]

The following layout test is timing out on all platforms of Chromium.

fast/forms/select-clientheight-large-size.html

See http://test-results.appspot.com/dashboards/flakiness_dashboard.html#tests=fast%2Fforms%2Fselect-clientheight-large-size.html

This test times out without printing any console message.

Probable cause:

This is a new test from r99653. Antaryami, do you have any idea why the test times out?

[Yuta Kitamura]

This test is timing out only on Debug bots. Release bots seem fine.

[Antaryami Pandia]

One probable reason might be that we are adding large number of option elements to the select box here (10000 to be precise).

I don't have chromium build.So can somebody with chromium build help me with this(by reducing the value), while I am setting up chromium build.

[Ojan Vafai]

I can fix this. Is there a reason we need to test 10000 options? Looking at r99653, testing 100 options seems like it would cover all the same edge cases.

[Ojan Vafai]

Created attachment 114342 [details]
Patch

[Tony Chang]

Comment on attachment 114342 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=114342&action=review

> LayoutTests/fast/forms/select-clientheight-large-size.html:23
> +        for (var i = 0; i<= numOptions; i++)

Nit: missing space between i and <=

[Ojan Vafai]

Committed r99759: <http://trac.webkit.org/changeset/99759>