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

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,

Re: [9] Review request for JDK-8161473:[TEST_BUG] [macosx] add autodelay to java/awt/TrayIcon/TrayIconEventModifiers/TrayIconEventModifiersTest.java

2016-10-10 Thread Anubhav Meena
Hi Ambarish, Have made the changes you suggested. Please find them here http://cr.openjdk.java.net/~rchamyal/anmeena/8161473/webrev.01/ Thanks, Anubhav > On Oct 10, 2016, at 2:01 PM, Ambarish Rapte wrote: > > Hi Anubhav, > > Along with the fix,

Re: [9] Review request for JDK-8161473:[TEST_BUG] [macosx] add autodelay to java/awt/TrayIcon/TrayIconEventModifiers/TrayIconEventModifiersTest.java

2016-10-10 Thread Ambarish Rapte
Hi Anubhav, Along with the fix, could you please add below trivial changes as well, 1. Add @key headful & @bug 8161473 jtreg tags 2. Change wild card * import to add specific package imports. 3. Update copyright with year 2016. Regards,