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/ <http://cr.openjdk.java.net/~mhalder/8158380/webrev.02/> Thanks, Manajit > On 10-Oct-2016, at 5:58 pm, Sergey Bylokhov <sergey.bylok...@oracle.com> > wrote: > > 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 Bylokhov >>> <sergey.bylok...@oracle.com <mailto:sergey.bylok...@oracle.com>> wrote: >>> >>> 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.net/browse/JDK-8158380 >>>> >>>> Webrev: >>>> http://cr.openjdk.java.net/~mhalder/8158380/webrev.00/ >>>> >>>> Issue: >>>> Regression: java/awt/List/ActionEventTest/ActionEventTest.java >>>> >>>> Cause: >>>> ActionEvent getModifiers() method was returning wrong value in test >>>> ActionEventTest.java due to wrong modifier value. >>>> Submenu was not activated by menmonic key press in SubMenuShowTest.java. >>>> In both the cases event queue was not processed completely for robot >>>> keyPress and keyRelease events. >>>> >>>> Fix: >>>> In both the cases problem was solved by adding waitForIdle after robot >>>> operations (keyPress and keyRelease). >>>> >>>> Regards, >>>> Manajit >>> >>> >>> -- >>> Best regards, Sergey. >> > > > -- > Best regards, Sergey.
