<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="font-family: -apple-system-font; line-height: 16px;" class=""><span style="color: rgb(0, 30, 87); font-family: Menlo;" class="">Hi Sergey,</span></div><div class=""><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><br class=""></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class="">Kindly review the fix for JDK9.</font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><br class=""></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><b class="">Bug</b>: </font></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8160266" class="">https://bugs.openjdk.java.net/browse/JDK-8160266</a></span></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><u class=""><br class=""></u></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><b class="">Webrev</b>: </font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""></div><div class=""><font face="-apple-system-font" class=""><span style="line-height: 16px;" class=""><a href="http://cr.openjdk.java.net/~mhalder/8160266/webrev.00/" class="">http://cr.openjdk.java.net/~mhalder/8160266/webrev.00/</a></span></font></div><div class=""><br class=""></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><b class="">Issue: </b></font></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class="">[macosx] NestedModalDialogTest.java and NestedModelessDialogTest.java tests does not run with current JDK codebase after taking the files from MACOSX_PORT</span></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><br class=""></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><b class="">Cause: </b></font></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class="">Both the tests were written using unit.framework.</span></font></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class=""><br class=""></span></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class=""><b class="">Fix: </b></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><span style="color: rgb(0, 30, 87); font-family: Menlo;" class="">Test files are modified to remove the dependency of junit.framework and related API calls. Test files were moved from MACOSX_PORT to the current JDK 9 codebase and added at the corresponding folder location:</span></div><div class=""><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class="">test/java/awt/Dialog/NestedDialogs/Modal/NestedModalDialogTest.java</span></font></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class="">test/java/awt/Dialog/NestedDialogs/Modeless/NestedModelessDialogTest.java</span></font></div></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><span style="color: rgb(0, 30, 87); font-family: Menlo;" class=""><br class=""></span></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class="">Along with the test file 3 supporting files are also moved. The files are:</span></font></div><div class=""><font color="#001e57" face="Menlo" class=""><span style="line-height: 16px;" class=""><div class="">test/java/awt/regtesthelpers/RobotUtilities.java</div><div class="">test/java/awt/regtesthelpers/VisibilityValidator.java</div><div class="">test/java/awt/regtesthelpers/Waypoint.java</div></span></font></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><div class=""><font face="Menlo" color="#001e57" class=""><br class=""></font></div><div class=""><font face="Menlo" color="#001e57" class=""><br class=""></font></div></div><div style="font-family: -apple-system-font; line-height: 16px;" class=""><font face="Menlo" color="#001e57" class="">Regards,<br class="">Manajit</font></div></div></body></html>