<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>Hi Semyon,</p>
    <p>the fix looks good to me,  but the test link is broken in this
      webrev.<br>
    </p>
    <pre class="moz-signature" cols="72">Thanks,
Alexander.</pre>
    <div class="moz-cite-prefix">On 30/10/2017 20:37, Semyon Sadetsky
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:a13dc787-9229-f08d-4237-870b2241792b@oracle.com">
      <meta http-equiv="content-type" content="text/html; charset=utf-8">
      <p> </p>
      <div class="moz-text-flowed" style="font-family: -moz-fixed;
        font-size: 12px;" lang="x-unicode">Hello, <br>
        <br>
        Please review fix for JDK10: <br>
        <br>
        bug: <a class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8190230"
          moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8190230</a>
        <br>
        <br>
        webrev: <a class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Essadetsky/8190230/webrev.00/"
          moz-do-not-send="true">http://cr.openjdk.java.net/~ssadetsky/8190230/webrev.00/</a>
        <br>
        <br>
        After the 8169589 fix siblings of the same parent are overlapped
        in the order they appear in  the array of child windows. But
        this order do not preserve the previous overlapping order in
        which the sibling windows have been shown before the order
        update. That is the cause of unexpected reorder. <br>
        <br>
        In the fix each platform window is attributed with a timestamp
        of the moment when it was show in front last time. Then this
        timestamp is used to sort the sibling windows array before
        putting them into an order to each other. <br>
        <br>
        --Semyon <br>
        <br>
      </div>
    </blockquote>
    <br>
  </body>
</html>