<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:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
@font-face
        {font-family:Consolas;
        panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;
        color:black;}
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.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;
        color:black;}
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;}
.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;}
/* List Definitions */
@list l0
        {mso-list-id:540242836;
        mso-list-type:hybrid;
        mso-list-template-ids:-1309926956 -136311462 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l0:level1
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:.75in;
        text-indent:-.25in;}
@list l0:level2
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:1.25in;
        text-indent:-.25in;}
@list l0:level3
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        margin-left:1.75in;
        text-indent:-9.0pt;}
@list l0:level4
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:2.25in;
        text-indent:-.25in;}
@list l0:level5
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:2.75in;
        text-indent:-.25in;}
@list l0:level6
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        margin-left:3.25in;
        text-indent:-9.0pt;}
@list l0:level7
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:3.75in;
        text-indent:-.25in;}
@list l0:level8
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        margin-left:4.25in;
        text-indent:-.25in;}
@list l0:level9
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        margin-left:4.75in;
        text-indent:-9.0pt;}
ol
        {margin-bottom:0in;}
ul
        {margin-bottom:0in;}
--></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-US link="#0563C1" vlink="#954F72"><div class=WordSection1><p class=MsoNormal><span style='color:#1F497D'>Hi Semyon,<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'>Aim of this testcase Is to check whether Frame state is “Frame.MAXIMIZED_BOTH” when Frame size is set to maximizedbounds of Frame.<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 scenario is true in Ubuntu and MAC Operating systems and not in other operating systems.<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'><snippet><o:p></o:p></span></p><p class=MsoNormal style='text-autospace:none'><span style='font-size:10.0pt;font-family:Consolas'>  </span><b><span style='font-size:10.0pt;font-family:Consolas;color:#7F0055'>boolean</span></b><span style='font-size:10.0pt;font-family:Consolas'> <u>frameStateSupported</u> = Toolkit.<i>getDefaultToolkit</i>().isFrameStateSupported(Frame.</span><b><i><span style='font-size:10.0pt;font-family:Consolas;color:#0000C0'>MAXIMIZED_BOTH</span></i></b><span style='font-size:10.0pt;font-family:Consolas'>);</span><span style='font-size:10.0pt;font-family:Consolas;color:windowtext'><o:p></o:p></span></p><p class=MsoNormal><span style='font-size:10.0pt;font-family:Consolas'>  System.</span><b><i><span style='font-size:10.0pt;font-family:Consolas;color:#0000C0'>out</span></i></b><span style='font-size:10.0pt;font-family:Consolas'>.println(</span><span style='font-size:10.0pt;font-family:Consolas;color:#6A3E3E'>frameStateSupported</span><span style='font-size:10.0pt;font-family:Consolas'>);<o:p></o:p></span></p><p class=MsoNormal><span style='font-size:10.0pt;font-family:Consolas'></snippet><o:p></o:p></span></p><p class=MsoNormal><span style='font-size:10.0pt;font-family:Consolas'><o:p> </o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Agreed that Above snippet returns <span style='background:yellow;mso-highlight:yellow'>true</span> even in windows.<o:p></o:p></span></p><p class=MsoNormal><span style='color:#1F497D'>Actual Scenario is While checking for Frame State when Maximized bounds is set (without explicitly setting Frame State), Windows and other Operating systems returns Frame.NORMAL state where as MAC and UBUNTU returns Frame.MAXIMIZED_BOTH state.</span><span style='font-size:10.0pt;font-family:Consolas'> <o:p></o:p></span></p><p class=MsoNormal><span style='font-size:10.0pt;font-family:Consolas'><o:p> </o:p></span></p><p class=MsoNormal><span style='font-size:10.0pt;font-family:Consolas'><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><span style='color:windowtext'>From:</span></b><span style='color:windowtext'> Semyon Sadetsky <br><b>Sent:</b> Wednesday, April 27, 2016 13:35<br><b>To:</b> Ramesh Gangadhar; 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 style='margin-bottom:12.0pt'>Hi Ramesh,<br><br>It looks like you tries to implement the Toolkit# isFrameStateSupported(state) method in your test.<br>This is toolkit's job just call it to detect is the required frame state is supported by the platform.<br><br>--Semyon<span style='font-size:12.0pt'><o:p></o:p></span></p><div><p class=MsoNormal>On 4/19/2016 8:40 AM, Ramesh Gangadhar wrote:<o:p></o:p></p></div><blockquote style='margin-top:5.0pt;margin-bottom:5.0pt'><p class=MsoNormal><span style='color:#1F497D'>Hello All,</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'>This is a gentle remainder for review.</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'>Best Regards,</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'>Ramesh BG</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></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'> </span><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:  <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></span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><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 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.</b><o:p></o:p></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;mso-list:l0 level1 lfo2'><![if !supportLists]><span style='mso-list:Ignore'>1.<span style='font:7.0pt "Times New Roman"'>       </span></span><![endif]>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;mso-list:l0 level1 lfo2'><![if !supportLists]><span style='mso-list:Ignore'>2.<span style='font:7.0pt "Times New Roman"'>       </span></span><![endif]>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;mso-list:l0 level1 lfo2'><![if !supportLists]><span style='mso-list:Ignore'>3.<span style='font:7.0pt "Times New Roman"'>       </span></span><![endif]>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;mso-list:l0 level1 lfo2'><![if !supportLists]><span style='mso-list:Ignore'>4.<span style='font:7.0pt "Times New Roman"'>       </span></span><![endif]>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;mso-list:l0 level1 lfo2'><![if !supportLists]><span style='mso-list:Ignore'>5.<span style='font:7.0pt "Times New Roman"'>       </span></span><![endif]>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'> </span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></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,</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'>                Please check the test fails on 64 bit Ubuntu 14.04 LTS.</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'>Thanks,</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'>Ambarish</span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></p><p class=MsoNormal><span style='color:#1F497D'> </span><o:p></o:p></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</span><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> <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/%7Ejdv/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></blockquote><p class=MsoNormal><span style='font-size:12.0pt;font-family:"Times New Roman",serif'><o:p> </o:p></span></p></div></body></html>