<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 12 (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:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
p.MsoAcetate, li.MsoAcetate, div.MsoAcetate
        {mso-style-priority:99;
        mso-style-link:"Balloon Text Char";
        margin:0in;
        margin-bottom:.0001pt;
        font-size:8.0pt;
        font-family:"Tahoma","sans-serif";}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {mso-style-priority:34;
        margin-top:0in;
        margin-right:0in;
        margin-bottom:0in;
        margin-left:.5in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri","sans-serif";}
span.EmailStyle18
        {mso-style-type:personal;
        font-family:"Calibri","sans-serif";
        color:windowtext;}
span.EmailStyle19
        {mso-style-type:personal;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
span.EmailStyle20
        {mso-style-type:personal;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
span.EmailStyle21
        {mso-style-type:personal;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
span.EmailStyle22
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
span.BalloonTextChar
        {mso-style-name:"Balloon Text Char";
        mso-style-priority:99;
        mso-style-link:"Balloon Text";
        font-family:"Tahoma","sans-serif";}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
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 lang=EN-US link="#0563C1" vlink="#954F72"><div class=WordSection1><p class=MsoNormal><span style='color:#1F497D'>Hi Ramesh,<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>                Fix looks good to me.<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Regards,<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Ambarish<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><div><div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal><b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'>From:</span></b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'> Ramesh Gangadhar <br><b>Sent:</b> Tuesday, April 19, 2016 11:10 AM<br><b>To:</b> awt-dev@openjdk.java.net<br><b>Subject:</b> Re: <AWT Dev> Review Request For 8129569 : Test java/awt/Frame/MaximizedToUnmaximized/MaximizedToUnmaximized.java fails<o:p></o:p></span></p></div></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><span style='color:#1F497D'>Hello All,<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>This is a gentle remainder for review.<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Best Regards,<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Ramesh BG<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><div><div style='border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal><b>From:</b> Ramesh Gangadhar <br><b>Sent:</b> Thursday, April 14, 2016 19:40<br><b>To:</b> <a href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a><br><b>Subject:</b> Re: <AWT Dev> Review Request For 8129569 : Test java/awt/Frame/MaximizedToUnmaximized/MaximizedToUnmaximized.java fails<o:p></o:p></p></div></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Hi,<o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal>Please review fix for bug 8129569 in JDK 9<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>BUG ID:  <a href="https://bugs.openjdk.java.net/browse/JDK-8129569">https://bugs.openjdk.java.net/browse/JDK-8129569</a><o:p></o:p></p><p class=MsoNormal>Webrev:  <span style='color:#1F497D'><a href="http://cr.openjdk.java.net/~arapte/ramesh/8129569/webrev.01/">http://cr.openjdk.java.net/~arapte/ramesh/8129569/webrev.01/</a><o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal>Issue:<o:p></o:p></p><p class=MsoNormal>While checking for Frame State when Maximum bounds is set in  Mac and Ubuntu Operating system sets state = Frame.MAXIMIZED_BOTH but this is not true in other operating system.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Fix:<o:p></o:p></p><p class=MsoNormal>Code modified to set state = Frame.NORMAL when maximum bounds is set to Frame in other operating systems apart from <b>Ubuntu and Mac.<o:p></o:p></b></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Added command “uname –v” to check whether operating system is Ubuntu or not since using alone ‘System.getProperty("os.name")  will not be able to decide operating system as it returns “Linux” for Ubuntu also.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Verification:<o:p></o:p></p><p class=MsoListParagraph style='margin-left:.75in;text-indent:-.25in'>1.<span style='font-size:7.0pt;font-family:"Times New Roman","serif"'>       </span>Windows 7 returns NORMAL state for frame when maximum bounds is set to Frame.<o:p></o:p></p><p class=MsoListParagraph style='margin-left:.75in;text-indent:-.25in'>2.<span style='font-size:7.0pt;font-family:"Times New Roman","serif"'>       </span>Oracle Linux 6.4 returns NORMAL state for frame when maximum bounds is set to Frame.<o:p></o:p></p><p class=MsoListParagraph style='margin-left:.75in;text-indent:-.25in'>3.<span style='font-size:7.0pt;font-family:"Times New Roman","serif"'>       </span>Solaris returns NORMAL state for frame when maximum bounds is set to Frame.<o:p></o:p></p><p class=MsoListParagraph style='margin-left:.75in;text-indent:-.25in'>4.<span style='font-size:7.0pt;font-family:"Times New Roman","serif"'>       </span>Ubuntu returns <b>MAXIMIZED_BOTH</b> state for frame when maximum bounds is set to Frame.<o:p></o:p></p><p class=MsoListParagraph style='margin-left:.75in;text-indent:-.25in'>5.<span style='font-size:7.0pt;font-family:"Times New Roman","serif"'>       </span>Mac returns <b>MAXIMIZED_BOTH</b> state for frame when maximum bounds is set to Frame<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>@Ambarish: Thank you for pointing to Ubuntu.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Best Regards,<o:p></o:p></p><p class=MsoNormal>Ramesh BG<o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><div><div style='border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal><b>From:</b> Ambarish Rapte <br><b>Sent:</b> Thursday, April 14, 2016 1:57 PM<br><b>To:</b> Ramesh Gangadhar; <a href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a><br><b>Subject:</b> RE: <AWT Dev> Review Request For 8129569 : Test java/awt/Frame/MaximizedToUnmaximized/MaximizedToUnmaximized.java fails<o:p></o:p></p></div></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><span style='color:#1F497D'>Hi Ramesh,<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>                Please check the test fails on 64 bit Ubuntu 14.04 LTS.<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Thanks,<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Ambarish<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'><o:p> </o:p></span></p><div><div style='border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0in 0in 0in'><p class=MsoNormal><b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'>From:</span></b><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'> Ramesh Gangadhar <br><b>Sent:</b> Thursday, April 14, 2016 12:04 PM<br><b>To:</b> </span><a href="mailto:awt-dev@openjdk.java.net"><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'>awt-dev@openjdk.java.net</span></a><span style='font-size:10.0pt;font-family:"Tahoma","sans-serif"'><br><b>Subject:</b> <AWT Dev> Review Request For 8129569 : Test java/awt/Frame/MaximizedToUnmaximized/MaximizedToUnmaximized.java fails<o:p></o:p></span></p></div></div><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Hi,<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Please review fix for bug 8129569 in JDK 9<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>BUG ID: <a href="https://bugs.openjdk.java.net/browse/JDK-8129569">https://bugs.openjdk.java.net/browse/JDK-8129569</a><o:p></o:p></p><p class=MsoNormal>Webrev: <a href="http://cr.openjdk.java.net/~jdv/Ramesh/8129569/webrev.00/">http://cr.openjdk.java.net/~jdv/Ramesh/8129569/webrev.00/</a><o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Issue:<o:p></o:p></p><p class=MsoNormal>While checking for Frame State when Maximum bounds is set Mac Operating system sets state = Frame.MAXIMIZED_BOTH but this is not true in other operating system.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Fix:<o:p></o:p></p><p class=MsoNormal>Code modified to set state = Frame.NORMAL when maximum bounds is set to Frame in other operating systems.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Best Regards,<o:p></o:p></p><p class=MsoNormal>Ramesh BG<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p></div></body></html>