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/ModalDialogActivationTest/ModalDialogActivationTest.java 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 guidelines. Regards, Ambarish