<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Looks good to me.<br>
    </p>
    <p>--Semyon<br>
    </p>
    <br>
    <div class="moz-cite-prefix">On 09/21/2017 01:44 AM, Pankaj Bansal
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:9e451163-a1d4-44c4-9484-43210aef032f@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:"Times New Roman \, serif";
        panose-1:0 0 0 0 0 0 0 0 0 0;}
/* 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
        {mso-style-priority:99;
        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.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-compose;
        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;}
--></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:#1F497D">Hi
            Sergey/Seymon,<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">I have
            incorporated the webrev for review comments.<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D">Yes Sergey, the
            test passes on Ubuntu with GTK L&F.<o:p></o:p></span></p>
        <p><span style="color:#1F497D">Webrev:  </span><a
href="http://cr.openjdk.java.net/%7Epsadhukhan/pankaj/JDK-6857809-webrev.01/"
            moz-do-not-send="true">http://cr.openjdk.java.net/~psadhukhan/pankaj/JDK-6857809-webrev.01/</a><span
            style="font-size:12.0pt"><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">Pankaj Bansal<o:p></o:p></span></p>
        <p class="MsoNormal"><span style="color:#1F497D"><o:p> </o:p></span></p>
        <p class="MsoNormal"><a name="_MailEndCompose"
            moz-do-not-send="true"><span style="color:#1F497D"><o:p> </o:p></span></a></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, September 20, 2017 10:43 PM<br>
                <b>To:</b> Pankaj Bansal; <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> [10] Review Request:
                JDK-6857809 : [TEST_BUG]
                java/awt/Component/PrintAllXcheckJNI/PrintAllXcheckJNI.java<o:p></o:p></span></p>
          </div>
        </div>
        <p class="MsoNormal"><o:p> </o:p></p>
        <p>On 09/20/2017 09:21 AM, Pankaj Bansal wrote:<span
            style="font-size:12.0pt"><o:p></o:p></span></p>
        <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
          <p class="MsoNormal"><span style="color:#1F497D">Hi Seymon,</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">I think the
              issue is because no size is set on frame. This is also the
              proposed fix in the bug description.</span><o:p></o:p></p>
        </blockquote>
        <p class="MsoNormal"><span
            style="font-size:12.0pt;font-family:"Times New
            Roman","serif"">This is good, but setting a
            fixed frame size forces the java Frame object to return that
            size only at the very beginning the real and final frame
            dimensions will be established  asynchronously by the native
            window subsystem.</span><span
            style="font-size:12.0pt;font-family:"Times New
            Roman","serif";color:#1F497D"><br>
            <br>
          </span><span style="font-size:12.0pt;font-family:"Times
            New Roman","serif""><o:p></o:p></span></p>
        <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
          <p class="MsoNormal"><span style="color:#1F497D">Regards,</span><o:p></o:p></p>
          <p class="MsoNormal"><span style="color:#1F497D">Pankaj</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><span style="color:windowtext">From:</span></b><span
                  style="color:windowtext"> Semyon Sadetsky <br>
                  <b>Sent:</b> Wednesday, September 20, 2017 9:02 PM<br>
                  <b>To:</b> Pankaj Bansal; <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> [10] Review
                  Request: JDK-6857809 : [TEST_BUG]
                  java/awt/Component/PrintAllXcheckJNI/PrintAllXcheckJNI.java</span><o:p></o:p></p>
            </div>
          </div>
          <p class="MsoNormal"> <o:p></o:p></p>
          <p>Hi Pankaj,<o:p></o:p></p>
          <p>The root cause of the issue is that the underling native
            window is not yet created when its with and height are
            requested. This is because windows are created
            asynchronously. The test should be fixed by adding
            Robot.waitForIdle() after frame.setVisible(true).<o:p></o:p></p>
          <p>--Semyon<o:p></o:p></p>
          <p class="MsoNormal"> <o:p></o:p></p>
          <div>
            <p class="MsoNormal">On 09/20/2017 05:49 AM, Pankaj Bansal
              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>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Please review the fix for test program
              test
              java/awt/Component/PrintAllXcheckJNI/PrintAllXcheckJNI.java
              for JDK 10.<o:p></o:p></p>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Bug:<o:p></o:p></p>
            <p class="MsoNormal"><a
                href="https://bugs.openjdk.java.net/browse/JDK-6857809"
                moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-6857809</a><o:p></o:p></p>
            <p class="MsoNormal"> <o:p></o:p></p>
            <p class="MsoNormal">Webrev:<o:p></o:p></p>
            <p class="MsoNormal"><span style="color:#1F497D"><a
                  href="http://cr.openjdk.java.net/%7Eaghaisas/pankaj/6857809/webrev.0/"
                  moz-do-not-send="true">http://cr.openjdk.java.net/~aghaisas/pankaj/6857809/webrev.0/</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">The test
               java/awt/Component/PrintAllXcheckJNI/PrintAllXcheckJNI.java
              fails on Linux intermittently due to
              “java.lang.IllegalArgumentException: Width (0) and height
              (0) cannot be <= 0” exception.<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">Made changes in
              java/awt/Component/PrintAllXcheckJNI/PrintAllXcheckJNI.java
              to set the size of frame to make the test more stable.<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">Pankaj Bansal<o:p></o:p></p>
          </blockquote>
          <p class="MsoNormal"><span
              style="font-size:12.0pt;font-family:"Times New Roman
              , serif","serif""> </span><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>
    </blockquote>
    <br>
  </body>
</html>