<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Still looks good.<br>
    <pre class="moz-signature" cols="72">Thanks,

Alexander.</pre>
    <div class="moz-cite-prefix">On 04/06/2015 03:46 PM, Semyon Sadetsky
      wrote:<br>
    </div>
    <blockquote cite="mid:5522802B.6050106@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      surely!<br>
      <br>
      webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Essadetsky/7042645/webrev.01/">http://cr.openjdk.java.net/~ssadetsky/7042645/webrev.01/</a><br>
      <br>
      --Semyon<br>
      <br>
      <div class="moz-cite-prefix">On 4/6/2015 3:33 PM, Alexander
        Zvegintsev wrote:<br>
      </div>
      <blockquote cite="mid:55227D23.1050608@oracle.com" type="cite">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        Hello Semyon,<br>
        <br>
        the fix looks good to me, <br>
        but could you please update the copyright year in all files and
        fix the
        <meta http-equiv="content-type" content="text/html;
          charset=utf-8">
        indentation of if statement in awt_Button.cpp before push?<br>
        <br>
        <pre class="moz-signature" cols="72">Thanks,

Alexander.</pre>
        <div class="moz-cite-prefix">On 04/06/2015 03:11 PM, Semyon
          Sadetsky wrote:<br>
        </div>
        <blockquote cite="mid:552277E2.9080907@oracle.com" type="cite">Hello,


          <br>
          <br>
          Please review fix for JDK9. <br>
          <br>
          webrev: <a moz-do-not-send="true"
            class="moz-txt-link-freetext"
            href="http://cr.openjdk.java.net/%7Essadetsky/7042645/webrev.00/">http://cr.openjdk.java.net/~ssadetsky/7042645/webrev.00/</a>
          <br>
          bug: <a moz-do-not-send="true" class="moz-txt-link-freetext"
            href="https://bugs.openjdk.java.net/browse/JDK-7042645">https://bugs.openjdk.java.net/browse/JDK-7042645</a>
          <br>
          <br>
          *ROOT CAUSE: <br>
          The assertion fails message triggered in awt_Button.cpp
          because a WinAPI function used to draw button focus rectangle
          returns zero. Tests run successfully on the release build just
          because assertions are ignored in it but they are in failure
          as well. <br>
          <br>
          MSDN documentation for the function return value: <br>
          <br>
          Return value <br>
          If the function succeeds, the return value is nonzero. <br>
          If the function fails, the return value is zero. <br>
          <br>
          is wrong, because it returns zero when window is not visible
          (for example, outside visible screen area). <br>
          The discrepancy reveals itself when the function returns zero
          which means error but subsequent ::GetLastError() call returns
          error code 0 which means successful execution. <br>
          <br>
          The same exists in awt_Checkbox.cpp and awt_Component.cpp <br>
          <br>
          *SOLUTION <br>
          ::GetlastError() returning error code is asserted to be 0 in
          case if ::DrawFocusRect() returns zero. <br>
          <br>
          *TESTING <br>
          This is a JCK test failure issue. No extra regression testing
          needed. <br>
          <br>
          --Semyon <br>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>