<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    +1<br>
    <br>
    -Phil.<br>
    <br>
    <div class="moz-cite-prefix">On 10/31/18 12:53 PM, Ambarish Rapte
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:5c3d42eb-6b95-464f-a0cf-f9e768df8437@default">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <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: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:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
pre
        {mso-style-priority:99;
        mso-style-link:"HTML Preformatted Char";
        margin:0in;
        margin-bottom:.0001pt;
        font-size:10.0pt;
        font-family:"Courier New";
        color:black;}
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;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;
        color:black;}
span.HTMLPreformattedChar
        {mso-style-name:"HTML Preformatted Char";
        mso-style-priority:99;
        mso-style-link:"HTML Preformatted";
        font-family:Consolas;
        color:black;}
span.new
        {mso-style-name:new;}
span.EmailStyle22
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle23
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle24
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.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:165439864;
        mso-list-template-ids:-1183952424;}
@list l1
        {mso-list-id:1424912694;
        mso-list-type:hybrid;
        mso-list-template-ids:-1254033006 67698703 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l1:level1
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l1:level2
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l1:level3
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
@list l1:level4
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l1:level5
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l1:level6
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
@list l1:level7
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l1:level8
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l1:level9
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        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]-->
      <div class="WordSection1">
        <p class="MsoNormal"><span style="color:windowtext">Thanks Phil
            & Sergey for review comments,<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:windowtext">Please take
            a look at updated webrev: <a
              href="http://cr.openjdk.java.net/~arapte/8198001/webrev.02/"
              moz-do-not-send="true">http://cr.openjdk.java.net/~arapte/8198001/webrev.02/</a><o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:windowtext"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span style="color:windowtext">Regards,<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:windowtext">Ambarish<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:windowtext"><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"> Phil Race <br>
                <b>Sent:</b> Wednesday, October 31, 2018 12:58 AM<br>
                <b>To:</b> Ambarish Rapte
                <a class="moz-txt-link-rfc2396E" href="mailto:ambarish.rapte@oracle.com"><ambarish.rapte@oracle.com></a>;
                <a class="moz-txt-link-abbreviated" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a><br>
                <b>Subject:</b> Re: <AWT Dev> [12] RFR :
                JDK-8198001 :
                java/awt/Menu/WrongParentAfterRemoveMenu/WrongParentAfterRemoveMenu.java
                debug assert on Windows<o:p></o:p></span></p>
          </div>
        </div>
        <p class="MsoNormal"><o:p> </o:p></p>
        <p class="MsoNormal"><o:p> </o:p></p>
        <pre><span class="new">+ * @bug 8165769, 8198001<o:p></o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new">Are commas allowed ?<o:p></o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new"><a href="http://openjdk.java.net/jtreg/tag-spec.html" moz-do-not-send="true">http://openjdk.java.net/jtreg/tag-spec.html</a><o:p></o:p></span></pre>
        <pre><o:p> </o:p></pre>
        <pre><span class="new">>Argument tokens are separated by whitespace; if commas are used,<o:p></o:p></span></pre>
        <pre><span class="new">>they will be considered to be part of the tokens to which they are adjacent.<o:p></o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new">So you now have a bug id of "8165769," acc. to these parsing rules.<o:p></o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new">So I think you should use just spaces <o:p></o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new">-phil.<o:p></o:p></span></pre>
        <div>
          <p class="MsoNormal">On 10/30/18 9:45 AM, Ambarish Rapte
            wrote:<o:p></o:p></p>
        </div>
        <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
          <p class="MsoNormal"><span style="color:windowtext">Hi Phil,</span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext">Thanks for
              the review comments.</span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext"> </span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext">Please
              take a look at updated webrev: <a
                href="http://cr.openjdk.java.net/~arapte/8198001/webrev.01/"
                moz-do-not-send="true">http://cr.openjdk.java.net/~arapte/8198001/webrev.01/</a></span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext">Updated
              the test with bud ID.</span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext">The test
              is not included in problem list.</span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext"> </span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext"> </span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext">Regards,</span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:windowtext">Ambarish</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><span style="color:windowtext">From:</span></b><span
                  style="color:windowtext"> Phil Race <br>
                  <b>Sent:</b> Tuesday, October 30, 2018 12:46 AM<br>
                  <b>To:</b> Ambarish Rapte <a
                    href="mailto:ambarish.rapte@oracle.com"
                    moz-do-not-send="true"><ambarish.rapte@oracle.com></a>;
                  <a href="mailto:awt-dev@openjdk.java.net"
                    moz-do-not-send="true">awt-dev@openjdk.java.net</a><br>
                  <b>Subject:</b> Re: <AWT Dev> [12] RFR :
                  JDK-8198001 :
                  java/awt/Menu/WrongParentAfterRemoveMenu/WrongParentAfterRemoveMenu.java
                  debug assert on Windows</span><o:p></o:p></p>
            </div>
          </div>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p class="MsoNormal" style="margin-bottom:12.0pt">The failing
            test should be updated with this bug id. Else it looks like
            a fix with no regression test.<br>
            Also if this was on the problem list (probably not if it
            only failed in debug builds) it<br>
            should be removed.<br>
            <br>
            -phil.<o:p></o:p></p>
          <div>
            <p class="MsoNormal">On 10/29/18 9:29 AM, Ambarish Rapte
              wrote:<o:p></o:p></p>
          </div>
          <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
            <p class="MsoNormal">Hi,<o:p></o:p></p>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Please review this windows only fix:<o:p></o:p></p>
            <p class="MsoNormal">JBS: <a
                href="https://bugs.openjdk.java.net/browse/JDK-8198001"
                moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8198001</a><o:p></o:p></p>
            <p class="MsoNormal">Webrev: <a
                href="http://cr.openjdk.java.net/~arapte/8198001/webrev.00/"
                moz-do-not-send="true">http://cr.openjdk.java.net/~arapte/8198001/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>
            <ol style="margin-top:0in" start="1" type="1">
              <li class="MsoNormal" style="mso-list:l1 level1 lfo3">Test
                asserts with debug build jdk, only on windows.<o:p></o:p></li>
              <li class="MsoNormal" style="mso-list:l1 level1 lfo3">This
                test fails when Menu is removed from MenuBar
                (WrongParentAfterRemoveMenu.java  :: Line no 100)<o:p></o:p></li>
              <li class="MsoNormal" style="mso-list:l1 level1 lfo3">Assert
                at Line no. 209, awt_MenuBar.cpp :: RedrawMenuBar()<o:p></o:p></li>
              <li class="MsoNormal" style="mso-list:l1 level1 lfo3">AwtMenuBar::DeleteItem()
                ->  AwtMenuBar::RedrawMenuBar()  => calls
                GetOwnerHWnd() which returns NULL.<o:p></o:p></li>
            </ol>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Fix:  Value returned by GetOwnerHWnd()
              should be checked for NULL value.<o:p></o:p></p>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Verification: All MenuBar tests pass.<o:p></o:p></p>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Regards,<o:p></o:p></p>
            <p class="MsoNormal">Ambarish<o:p></o:p></p>
          </blockquote>
          <p class="MsoNormal"> <o:p></o:p></p>
        </blockquote>
        <p class="MsoNormal"><o:p> </o:p></p>
      </div>
    </blockquote>
    <br>
  </body>
</html>