<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <p>Hi Dmitry,</p>
    <p>Thanks for your reply. Please see my reply inline.</p>
    <p>Thanks,<br>
      Manajit<br>
    </p>
    <br>
    <div class="moz-cite-prefix">On 01/09/18 12:02 AM, Dmitry Markov
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:D7CF190A-D580-49F6-9E6E-FE65014E18C8@oracle.com">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      Hi Manajit,
      <div class=""><br class="">
      </div>
      <div class="">The current implementation assumes that
        orderAboveSiblings() places the window in front of other windows
        located at the same level. The proposed fix introduces new
        behaviour: if the window does not have an owner and child
        windows it won’t be ordered, (i.e. in certain conditions the
        window may be obscured by other windows even after
        orderAboveSibling() execution). Most likely such approach will
        break existed functionality - some windows will be incorrectly
        placed behind other windows.</div>
    </blockquote>
        If I am not wrong windows (other than Window.Type.POPUP) are
    already ordered in setVisible method at line no 632 while creation.
    While debugging I observed that orderFront is called twice when the
    window is displayed for the first time (in method setVisible and in
    method orderAboveSiblings). Now when Key press COMMAND + ` is
    pressed the current window receives windowDidBecomeMain notification
    and which calls orderFront and also COMMAND + ` tries to order the
    window above. Two time ordering the window when COMMAND + ` is
    pressed is causing problem in case of multiple windows.<br>
    <blockquote type="cite"
      cite="mid:D7CF190A-D580-49F6-9E6E-FE65014E18C8@oracle.com">
      <div class=""><br class="">
      </div>
      <div class="">I am sorry, but the relationship between the
        original problem described in the bug, (i.e. cycling through
        windows issue) and implementation of orderAboveSiblings() is NOT
        clear. Can you explain this?</div>
    </blockquote>
        This issue is a regression of JDK-8169589 introduced in JDK
    release 8u131. 8169589 introduced new window ordering model and
    which has introduced the cycling through window issue.<br>
    <blockquote type="cite"
      cite="mid:D7CF190A-D580-49F6-9E6E-FE65014E18C8@oracle.com">
      <div class=""><br class="">
      </div>
      <div class="">Thanks,</div>
      <div class="">Dmitry  <br class="">
        <div><br class="">
          <blockquote type="cite" class="">
            <div class="">On 31 Aug 2018, at 07:55, Manajit Halder <<a
                href="mailto:manajit.halder@oracle.com" class=""
                moz-do-not-send="true">manajit.halder@oracle.com</a>>
              wrote:</div>
            <br class="Apple-interchange-newline">
            <div class="">
              <meta http-equiv="content-type" content="text/html;
                charset=utf-8" class="">
              <div text="#000000" bgcolor="#FFFFFF" class="">
                <p class="">Hi All,<br class="">
                  <br class="">
                  Please review the fix for JDK12.<br class="">
                  <br class="">
                  <span style="font-family:"MS Gothic""
                    class="">

</span>Bug: <br class="">
                       <a moz-do-not-send="true"
                    href="https://bugs.openjdk.java.net/browse/JDK-8206392"
                    class="">https://bugs.openjdk.java.net/browse/JDK-8206392</a></p>
                <p class=""><a moz-do-not-send="true"
                    href="https://bugs.openjdk.java.net/browse/JDK-8206392"
                    class=""> </a><span style="font-family:"MS
                    Gothic"" class="">
</span>Webrev: <span
                    style="font-family:"MS Gothic"" class="">
</span><br
                    class="">
                      <a
                    href="http://cr.openjdk.java.net/%7Emhalder/8206392/webrev.00/"
                    moz-do-not-send="true" class="">http://cr.openjdk.java.net/~mhalder/8206392/webrev.00/</a></p>
                <p class="">Fix: <span style="font-family:"MS
                    Gothic"" class="">
</span><br class="">
                      Window ordering is not required if the window
                  doesn't own any other windows.</p>
                Regards,<br class="">
                Manajit </div>
            </div>
          </blockquote>
        </div>
        <br class="">
      </div>
    </blockquote>
    <br>
  </body>
</html>