<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html;
      charset=windows-1252">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    I would like to refer back to a comment you made in the previous fix<br>
<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>
    <br>
    > It is not mentioned in the focus spec whether the unfocusable
    maximized frames should be resizable or not.<br>
    <br>
    Yet there seems to be a JCK test that says it should not be
    resizable ?<br>
    <br>
    Can you review the spec. again ?<br>
    JCK must have based the test on something .. else the test is not
    valid.<br>
    <br>
    If we want that behaviour specified .. we should be specifying it ..<br>
    But I am not sure if it is actually enforceable on all window
    managers / desktops.<br>
    <br>
    But I have the same issue with this fix as the previous one. Perhaps
    not the fix,<br>
    but the explanation. The code being changed can't be understood
    without seeing<br>
    the method it calls, and the native method it in turn calls.<br>
    <br>
    Can you provide a detailed explanation as to how this change
    propagates down<br>
    and does the right thing ?<br>
    <br>
    BTW stylistically - if this is the right fix - you could do :<br>
    <br>
    <pre><span class="removed">setStyleBits(SHOULD_BECOME_KEY | SHOULD_BECOME_MAIN | ((isFocusable) ? RESIZABLE : 0), isFocusable);</span></pre>
    <br>
    -phil.<br>
    <br>
    <div class="moz-cite-prefix">On 06/14/2018 11:37 PM, Manajit Halder
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:6B14FB04-42F7-4344-80BA-67FF2EF3918A@oracle.com">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <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>
  </body>
</html>