Re: Review request for JDK-8166015: [PIT][TEST_BUG] stray character in java/awt/Focus/ModalDialogActivationTest/ModalDialogActivationTest.java

2016-09-19 Thread Semyon Sadetsky
][TEST_BUG] stray character in java/awt/Focus/ModalDialogActivationTest/ModalDialogActivationTest.java Hi, Please review this test bug fix for JDK9, Webrev: http://cr.openjdk.java.net/~arapte/8166015/webrev.00/ <http://cr.openjdk.java.net/%7Earapte/8166015/webrev.00/> Bug:

Re: Review request for JDK-8166015: [PIT][TEST_BUG] stray character in java/awt/Focus/ModalDialogActivationTest/ModalDialogActivationTest.java

2016-09-19 Thread Ajit Ghaisas
The fix looks good. Regards, Ajit From: Ambarish Rapte Sent: Monday, September 19, 2016 4:15 PM To: Sergey Bylokhov; Semyon Sadetsky; Ajit Ghaisas; awt-dev@openjdk.java.net Subject: Review request for JDK-8166015: [PIT][TEST_BUG] stray character in java/awt/Focus

Review request for JDK-8166015: [PIT][TEST_BUG] stray character in java/awt/Focus/ModalDialogActivationTest/ModalDialogActivationTest.java

2016-09-19 Thread Ambarish Rapte
Hi, Please review this test bug fix for JDK9, Webrev: http://cr.openjdk.java.net/~arapte/8166015/webrev.00/ Bug: https://bugs.openjdk.java.net/browse/JDK-8166015 Issue:An unwanted character causes compiler error. Fix: Removed the character & did few small corrections as