The fix looks good to me.

Thanks,
Alexandr.

On 10/17/2016 9:46 AM, Semyon Sadetsky wrote:

Looks good.

--Semyon


On 10/17/2016 5:54 AM, Ambarish Rapte wrote:

Hi,

                Please review this test bug fix,

                Bug: https://bugs.openjdk.java.net/browse/JDK-8167288

Webrev: http://cr.openjdk.java.net/~arapte/8167288/webrev.00 <http://cr.openjdk.java.net/%7Earapte/8167288/webrev.00>

Issue:

The test creates a child process to display a warning window.

                That child process is expected to end within 5 seconds.

But on some slower or busy system, child process takes more time to end which causes failure.

Fix:

                Fix is to wait more time for child process to end.

Now the test will wait for maximum of 10 seconds in iteration of 2 seconds.

Regards,

Ambarish



Reply via email to