Re: RFR JDK-8242526: PIT: javax/swing/JInternalFrame/8020708/bug8020708.java fails in mach5 ubuntu system

2020-04-13 Thread Pankaj Bansal
Look good to me -Pankaj On 11/04/20 9:46 PM, Prasanta Sadhukhan wrote: On 11-Apr-20 8:08 PM, Sergey Bylokhov wrote: Hi, Prasanta. Not sure that "waitForIdle()" should be removed at line 107, it waits till the moment the window became visible, and w/o it the Util.getCenterPoint may return

Re: RFR JDK-8242526: PIT: javax/swing/JInternalFrame/8020708/bug8020708.java fails in mach5 ubuntu system

2020-04-11 Thread Sergey Bylokhov
+1 On 4/11/20 9:16 am, Prasanta Sadhukhan wrote: On 11-Apr-20 8:08 PM, Sergey Bylokhov wrote: Hi, Prasanta. Not sure that "waitForIdle()" should be removed at line 107, it waits till the moment the window became visible, and w/o it the Util.getCenterPoint may return wrong coordinates. It is

Re: RFR JDK-8242526: PIT: javax/swing/JInternalFrame/8020708/bug8020708.java fails in mach5 ubuntu system

2020-04-11 Thread Prasanta Sadhukhan
On 11-Apr-20 8:08 PM, Sergey Bylokhov wrote: Hi, Prasanta. Not sure that "waitForIdle()" should be removed at line 107, it waits till the moment the window became visible, and w/o it the Util.getCenterPoint may return wrong coordinates. It is also unclear how the robot.delay(500); after the

Re: RFR JDK-8242526: PIT: javax/swing/JInternalFrame/8020708/bug8020708.java fails in mach5 ubuntu system

2020-04-11 Thread Sergey Bylokhov
Hi, Prasanta. Not sure that "waitForIdle()" should be removed at line 107, it waits till the moment the window became visible, and w/o it the Util.getCenterPoint may return wrong coordinates. It is also unclear how the robot.delay(500); after the "hitKeys" will help, since the buttons were pre

RFR JDK-8242526: PIT: javax/swing/JInternalFrame/8020708/bug8020708.java fails in mach5 ubuntu system

2020-04-11 Thread Prasanta Sadhukhan
Hi All, Please review a test fix for an issue seen in PIT where the test is failing on ubuntu 18.04 on mach5 system The previous 8226230 fix fixes the issue in local ubuntu18.04 but it seems it sometimes still fail in mach5 ubuntu system. Added some delay to ensure it passes in mach5 system