<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=us-ascii"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:-apple-system-font;
        panose-1:0 0 0 0 0 0 0 0 0 0;}
@font-face
        {font-family:Menlo;
        panose-1:0 0 0 0 0 0 0 0 0 0;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0cm;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman",serif;
        color:black;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body bgcolor=white lang=EN-IN link=blue vlink=purple><div class=WordSection1><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Looks fine to me.<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Regards,<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'>Rajeev Chamyal<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US'><o:p> </o:p></span></p><div><div style='border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm'><p class=MsoNormal><b><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:windowtext'>From:</span></b><span lang=EN-US style='font-size:11.0pt;font-family:"Calibri",sans-serif;color:windowtext'> Alexandr Scherbatiy <br><b>Sent:</b> 14 July 2016 20:45<br><b>To:</b> Manajit Halder<br><b>Cc:</b> awt-dev@openjdk.java.net; Avik Niyogi<br><b>Subject:</b> Re: <AWT Dev> <AWT dev>[9] Review request for JDK-8160266: [macosx] NestedModalDialogTest.java and NestedModelessDialogTest.java tests does not run with current JDK codebase after taking the files from MACOSX_PORT<o:p></o:p></span></p></div></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal style='margin-bottom:12.0pt'>The fix looks good to me.<br><br>Thanks,<br>Alexandr.<o:p></o:p></p><div><p class=MsoNormal>On 7/12/2016 2:47 PM, Manajit Halder wrote:<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt'><p class=MsoNormal>Hi Alexandr, <o:p></o:p></p><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Thank you for the review comments. Code is modified as per you suggestion. RobotUtilities class is replaced with direct calls to corresponding methods in Robot class and also Util class is used in some places.<o:p></o:p></p></div><div><p class=MsoNormal>Please review the modified code.<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><a href="http://cr.openjdk.java.net/%7Emhalder/8160266/webrev.01/">http://cr.openjdk.java.net/~mhalder/8160266/webrev.01/</a><o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Thanks,<o:p></o:p></p></div><div><p class=MsoNormal>Manajit<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt'><div><p class=MsoNormal>On 08-Jul-2016, at 3:57 pm, Alexandr Scherbatiy <<a href="mailto:alexandr.scherbatiy@oracle.com">alexandr.scherbatiy@oracle.com</a>> wrote:<o:p></o:p></p></div><p class=MsoNormal><o:p> </o:p></p><div><div><p class=MsoNormal style='margin-bottom:12.0pt'><br>Is it possible to get rid of the RobotUtilities class? Methods like RobotUtilities.pressKey(int num)/releaseKey(int num) are just direct calls to the corresponding methods in Robot.<br>Other necessary methods can be added to  test/java/awt/regtesthelpers/Util.java<br><br>Thanks,<br>Alexandr.<o:p></o:p></p><div><p class=MsoNormal>On 7/7/2016 1:32 PM, Manajit Halder wrote:<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt'><p class=MsoNormal>Hi All, <o:p></o:p></p><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Gentle remainder. Please review the fix for issue <a href="https://bugs.openjdk.java.net/browse/JDK-8160266">https://bugs.openjdk.java.net/browse/JDK-8160266</a>.<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal>Thanks,<o:p></o:p></p></div><div><p class=MsoNormal>Manajit<o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p><div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt'><div><p class=MsoNormal>On 24-Jun-2016, at 8:53 pm, Manajit Halder <<a href="mailto:manajit.halder@oracle.com">manajit.halder@oracle.com</a>> wrote:<o:p></o:p></p></div><p class=MsoNormal><o:p> </o:p></p><div><div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif;color:#001E57'>Hi Sergey,</span><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"-apple-system-font",serif'><o:p> </o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif;color:#001E57'>Kindly review the fix for JDK9.</span><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"-apple-system-font",serif'><o:p> </o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><b><span style='font-family:"Menlo",serif;color:#001E57'>Bug</span></b><span style='font-family:"Menlo",serif;color:#001E57'>: </span><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal><span style='font-family:"Menlo",serif;color:#001E57'><a href="https://bugs.openjdk.java.net/browse/JDK-8160266">https://bugs.openjdk.java.net/browse/JDK-8160266</a></span><o:p></o:p></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"-apple-system-font",serif'><o:p> </o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><b><span style='font-family:"Menlo",serif;color:#001E57'>Webrev</span></b><span style='font-family:"Menlo",serif;color:#001E57'>: </span><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal><span style='font-family:"-apple-system-font",serif'><a href="http://cr.openjdk.java.net/%7Emhalder/8160266/webrev.00/">http://cr.openjdk.java.net/~mhalder/8160266/webrev.00/</a></span><o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal style='line-height:12.0pt'><b><span style='font-family:"Menlo",serif;color:#001E57'>Issue: </span></b><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal><span style='font-family:"Menlo",serif;color:#001E57'>[macosx] NestedModalDialogTest.java and NestedModelessDialogTest.java tests does not run with current JDK codebase after taking the files from MACOSX_PORT</span><o:p></o:p></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"-apple-system-font",serif'><o:p> </o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><b><span style='font-family:"Menlo",serif;color:#001E57'>Cause: </span></b><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal><span style='font-family:"Menlo",serif;color:#001E57'>Both the tests were written using unit.framework.</span><o:p></o:p></p></div><div><p class=MsoNormal><o:p> </o:p></p></div><div><p class=MsoNormal style='line-height:12.0pt'><b><span style='font-family:"Menlo",serif;color:#001E57'>Fix: </span></b><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif'>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><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><div><p class=MsoNormal><span style='font-family:"Menlo",serif;color:#001E57'>test/java/awt/Dialog/NestedDialogs/Modal/NestedModalDialogTest.java</span><o:p></o:p></p></div><div><p class=MsoNormal><span style='font-family:"Menlo",serif;color:#001E57'>test/java/awt/Dialog/NestedDialogs/Modeless/NestedModelessDialogTest.java</span><o:p></o:p></p></div></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif'><br><br></span><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div><div><p class=MsoNormal><span style='font-family:"Menlo",serif;color:#001E57'>Along with the test file 3 supporting files are also moved. The files are:</span><o:p></o:p></p></div><div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif;color:#001E57'>test/java/awt/regtesthelpers/RobotUtilities.java<o:p></o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif;color:#001E57'>test/java/awt/regtesthelpers/VisibilityValidator.java<o:p></o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif;color:#001E57'>test/java/awt/regtesthelpers/Waypoint.java<o:p></o:p></span></p></div></div><div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"-apple-system-font",serif'><o:p> </o:p></span></p></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"-apple-system-font",serif'><o:p> </o:p></span></p></div></div><div><p class=MsoNormal style='line-height:12.0pt'><span style='font-family:"Menlo",serif;color:#001E57'>Regards,<br>Manajit</span><span style='font-family:"-apple-system-font",serif'><o:p></o:p></span></p></div></div></div></div></blockquote></div><p class=MsoNormal><o:p> </o:p></p></div></blockquote><p class=MsoNormal><o:p> </o:p></p></div></div></blockquote></div><p class=MsoNormal><o:p> </o:p></p></div></blockquote><p class=MsoNormal><o:p> </o:p></p></div></body></html>