Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-13 Thread Ajit Ghaisas
The fix looks good. Regards, Ajit -Original Message- From: Sergey Bylokhov Sent: Wednesday, October 12, 2016 8:12 PM To: Manajit Halder Cc: awt-dev@openjdk.java.net Subject: Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java Looks

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-12 Thread Sergey Bylokhov
Looks fine. On 12.10.16 14:40, Manajit Halder wrote: Thanks you for your comment. Copyright is corrected in both the tests. Please review the modified webrev. http://cr.openjdk.java.net/~mhalder/8158380/webrev.02/ Thanks, Manajit On 10-Oct-2016, at 5:58 pm, Sergey Bylokhov mailto:sergey.bylo

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-12 Thread Manajit Halder
Thanks you for your comment. Copyright is corrected in both the tests. Please review the modified webrev. http://cr.openjdk.java.net/~mhalder/8158380/webrev.02/ Thanks, Manajit > On 10-Oct-2016, at 5:58 pm, Sergey Bylokhov > wrote: >

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-10 Thread Sergey Bylokhov
It seems that "Copyright (c)" is incorrectly updated in both tests. On 10.10.16 14:41, Manajit Halder wrote: Hi Sergey, Thank you for the review. Please review the modified webrev. http://cr.openjdk.java.net/~mhalder/8158380/webrev.01/ Thanks, Manajit On 07-Oct-2016, at 7:41 pm, Sergey Bylok

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-10 Thread Manajit Halder
Hi Sergey, Thank you for the review. Please review the modified webrev. http://cr.openjdk.java.net/~mhalder/8158380/webrev.01/ Thanks, Manajit > On 07-Oct-2016, at 7:41 pm, Sergey Bylokhov > wrote: > > Hi, Manajit. > Instead of adding

Re: [9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-07 Thread Sergey Bylokhov
Hi, Manajit. Instead of adding these methods between operations you can call these after robot creation: Robot.setAutoDelay(int ms) Robot.setAutoWaitForIdle(boolean isOn) On 07.10.16 15:10, Manajit Halder wrote: Hi All, Kindly review the fix for JDK9. Bug: https://bugs.openjdk.java.n

[9] Review request for JDK-8158380: Regression: java/awt/List/ActionEventTest/ActionEventTest.java

2016-10-07 Thread Manajit Halder
Hi All, Kindly review the fix for JDK9. Bug: https://bugs.openjdk.java.net/browse/JDK-8158380 Webrev: http://cr.openjdk.java.net/~mhalder/8158380/webrev.00/ Issue: Regression: java/a