<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi Phil,<div class=""><br class=""></div><div class="">Please find my answer inline to your comment.</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 15-Jun-2018, at 11:24 PM, Phil Race <<a href="mailto:philip.race@oracle.com" class="">philip.race@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252" class="">
  
  <div text="#000000" bgcolor="#FFFFFF" class="">
    I would like to refer back to a comment you made in the previous fix<br class="">
<a class="moz-txt-link-freetext" href="http://mail.openjdk.java.net/pipermail/awt-dev/2018-February/013626.html">http://mail.openjdk.java.net/pipermail/awt-dev/2018-February/013626.html</a><br class="">
    <br class="">
    > It is not mentioned in the focus spec whether the unfocusable
    maximized frames should be resizable or not.<br class="">
    <br class="">
    Yet there seems to be a JCK test that says it should not be
    resizable ?<br class="">
    <br class="">
    Can you review the spec. again ?<br class="">
    JCK must have based the test on something .. else the test is not
    valid.<br class=""></div></div></blockquote><div><br class=""></div><span style="font-family: Menlo; font-size: 11px; background-color: rgb(255, 255, 255);" class="">Yes, I checked FocusSpec.html and it</span> doesn’t specify anything about resizable behaviour of non-focusable Frame.</div><div><br class=""></div><div>The <span style="font-family: Menlo; font-size: 11px; background-color: rgb(255, 255, 255);" class="">UnfocusableMaximizedFrameResizablity.java</span>  test passes on Window and Linux and fails on Mac OS. </div><div>Fix for issue 7158623 was done accordingly to make sure the behaviour is same on all platforms.<span style="font-family: Menlo; font-size: 11px; background-color: rgb(255, 255, 255);" class=""> </span></div><div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><span style="font-variant-ligatures: no-common-ligatures" class=""><br class=""></span></div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class="">If this behaviour is not correct then Window and Linux code should be changed accordingly so that all three platforms behave same.</div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><br class=""></div></div><div><blockquote type="cite" class=""><div class=""><div text="#000000" bgcolor="#FFFFFF" class=""><br class="">
    If we want that behaviour specified .. we should be specifying it ..<br class="">
    But I am not sure if it is actually enforceable on all window
    managers / desktops.<br class="">
    <br class="">
    But I have the same issue with this fix as the previous one. Perhaps
    not the fix,<br class="">
    but the explanation. The code being changed can't be understood
    without seeing<br class="">
    the method it calls, and the native method it in turn calls.<br class="">
    <br class="">
    Can you provide a detailed explanation as to how this change
    propagates down<br class="">
    and does the right thing ?<br class="">
    <br class=""></div></div></blockquote>The call flow:</div><div><br class=""></div><div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class="">updateFocusableWindowState() calls setStyleBits with style bits to be set on the window.</div><div style="margin: 0px; font-size: 11px; line-height: normal; font-family: Menlo; background-color: rgb(255, 255, 255);" class=""><div style="margin: 0px; line-height: normal;" class="">setStyleBits() calls native method nativeSetNSWindowStyleBits. nativeSetNSWindowStyleBits passes mask and 0 as the second parameter in our case (for non-focusable windows).</div><div style="margin: 0px; line-height: normal;" class=""><div style="margin: 0px; line-height: normal;" class="">Java_sun_lwawt_macosx_CPlatformWindow_nativeSetNSWindowStyleBits in AWTWindow.m generates newBits and applies it on the NSWindow.</div><div style="margin: 0px; line-height: normal;" class=""><br class=""></div><div style="margin: 0px; line-height: normal;" class="">My previous fix for issue 7158623 explains bits set on the window.</div><div style="margin: 0px; line-height: normal;" class=""><a href="http://openjdk.5641.n7.nabble.com/lt-AWT-Dev-gt-Subject-lt-AWT-dev-gt-11-Review-request-for-JDK-7158623-macosx-Should-an-unfocusable-m-td326691.html#a329071" class="">http://openjdk.5641.n7.nabble.com/lt-AWT-Dev-gt-Subject-lt-AWT-dev-gt-11-Review-request-for-JDK-7158623-macosx-Should-an-unfocusable-m-td326691.html#a329071</a></div></div></div><div><br class=""></div><br class=""><blockquote type="cite" class=""><div class=""><div text="#000000" bgcolor="#FFFFFF" class="">
    BTW stylistically - if this is the right fix - you could do :<br class="">
    <br class="">
    <pre class=""><span class="removed">setStyleBits(SHOULD_BECOME_KEY | SHOULD_BECOME_MAIN | ((isFocusable) ? RESIZABLE : 0), isFocusable);</span></pre>
    </div></div></blockquote>Changed the code as per the suggestion. Please review the modified code.</div><div><a href="http://cr.openjdk.java.net/~mhalder/8204860/webrev.01/" class="">http://cr.openjdk.java.net/~mhalder/8204860/webrev.01/</a></div><div><br class=""></div><div>Regards,</div><div>Manajit</div><div><br class=""><blockquote type="cite" class=""><div class=""><div text="#000000" bgcolor="#FFFFFF" class="">
    -phil.<br class="">
    <br class="">
    <div class="moz-cite-prefix">On 06/14/2018 11:37 PM, Manajit Halder
      wrote:<br class="">
    </div>
    <blockquote type="cite" cite="mid:6B14FB04-42F7-4344-80BA-67FF2EF3918A@oracle.com" class="">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252" class="">
      <div style="margin: 0px; line-height: normal; font-family:
        'Helvetica Neue'; color: rgb(69, 69, 69);" class="">Hi All,</div>
      <div style="margin: 0px; line-height: normal; font-family:
        'Helvetica Neue'; color: rgb(69, 69, 69); min-height: 14px;" class=""><br class="">
      </div>
      <div style="margin: 0px; line-height: normal; font-family:
        'Helvetica Neue'; color: rgb(69, 69, 69);" class="">Kindly
        review the fix for JDK11.</div>
      <div style="margin: 0px; line-height: normal; font-family:
        'Helvetica Neue'; color: rgb(69, 69, 69); min-height: 14px;" class=""><br class="">
      </div>
      <div style="margin: 0px; line-height: normal; font-family:
        'Helvetica Neue'; color: rgb(69, 69, 69);" class="">Bug: </div>
      <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><a href="https://bugs.openjdk.java.net/browse/JDK-8204860" class="" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8204860</a></font></div>
      <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><br class="">
        </font></div>
      <div style="margin: 0px; line-height: normal;" class="">
        <div style="margin: 0px; line-height: normal; font-family:
          'Helvetica Neue'; color: rgb(69, 69, 69);" class="">Webrev:</div>
        <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><a href="http://cr.openjdk.java.net/%7Emhalder/8204860/webrev.00/" class="" moz-do-not-send="true">http://cr.openjdk.java.net/~mhalder/8204860/webrev.00/</a></font></div>
        <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><br class="">
          </font></div>
        <div style="margin: 0px; line-height: normal;" class="">
          <div style="margin: 0px; line-height: normal; font-family:
            'Helvetica Neue'; color: rgb(69, 69, 69);" class="">Fix:</div>
          <div style="margin: 0px; line-height: normal;" class="">
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545">Frame is
                focusable:</font></div>
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><span class="Apple-tab-span" style="white-space:pre">  </span>Retaining
                the existing frame resizable behaviour (Fixes the
                current issue)</font><span style="color: rgb(69, 69,
                69); font-family: 'Helvetica Neue';" class="">.</span></div>
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545">Frame is
                non-focusable:</font></div>
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><span class="Apple-tab-span" style="white-space: pre;">        </span>Making
                the Frame non-resizable (Fix for issue 7158623).</font></div>
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545"><br class="">
              </font></div>
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545">Regards,</font></div>
            <div style="margin: 0px; line-height: normal;" class=""><font class="" face="Helvetica Neue" color="#454545">Manajit</font></div>
          </div>
        </div>
      </div>
    </blockquote>
    <br class="">
  </div>

</div></blockquote></div><br class=""></div></body></html>