And also, I think this test has nothing to do with "print". It should be moved to java/awt/Paint folder, I guess.

Do you have an idea why it is taking more than 20 secs to complete, which api is taking so much time? Is it on all platform, or some specific?

Regards
Prasanta
On 1/4/2017 4:20 PM, Ajit Ghaisas wrote:

Change looks fine.

Some correction can be done by replacing wildcard imports by specific imports.

Regards,

Ajit

*From:*Prem Balakrishnan
*Sent:* Wednesday, January 04, 2017 4:09 PM
*To:* Ajit Ghaisas; Prasanta Sadhukhan; awt-dev@openjdk.java.net
*Subject:* Review Request: JDK-8172009 [TEST_BUG] increase timeout in java/awt/print/PaintSetEnabledDeadlock/PaintSetEnabledDeadlock.java

Hi*,*

Please review fix for JDK 9,

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

*Webrev:*http://cr.openjdk.java.net/~pkbalakr/8172009/webrev.00/ <http://cr.openjdk.java.net/%7Epkbalakr/8172009/webrev.00/>

Test fails because it was taking more than 20 seconds to complete, updated timeout to 30 seconds.

Regards,

Prem


Reply via email to