Re: Review reuest for 8146168: [TEST_BUG] instability of java/awt/Frame/MaximizedToMaximized/MaximizedToMaximized.java

2015-12-29 Thread Sergey Bylokhov
Hi, Ambarish. Did you try to check why Robot.waitForIdle() does not work in this case?(in jdk9 it is implemented via (SunToolkit.realSync()) On 29/12/15 15:49, Ambarish Rapte wrote: Hi Rajeev, Thanks for the review, Please check the updated test file as per the review comments, http://cr.op

Re: Review reuest for 8146168: [TEST_BUG] instability of java/awt/Frame/MaximizedToMaximized/MaximizedToMaximized.java

2015-12-29 Thread Ambarish Rapte
Hi Rajeev, Thanks for the review, Please check the updated test file as per the review comments, http://cr.openjdk.java.net/~arapte/8146168/webrev.01/ Regards, Ambarish From: Rajeev Chamyal Sent: Tuesday, December 29, 2015 4:36 PM To: Ambarish Rapte; Semyon Sadetsky; Prasanta Sadhukhan

Re: Review reuest for 8146168: [TEST_BUG] instability of java/awt/Frame/MaximizedToMaximized/MaximizedToMaximized.java

2015-12-29 Thread Rajeev Chamyal
Hello Ambarish, 1) Please mark frameDisplayed & frameMaximized as volatile as these are accessed from different threads. 2) Instead of disposing frame multiple times you can add a finally block. 3) I think you can also remove the @author tag from test. Regards, Rajeev Cha