<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    +1<br>
    <br>
    -phil.<br>
    <br>
    On 8/8/18, 5:56 AM, Manajit Halder wrote:
    <blockquote
      cite="mid:3694bb5c-1c08-cd3f-ba60-b6cd97e73f41@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
      <p>Hi Phil,<br>
        <br>
        Thanks for your review comment. <br>
        <br>
        Reverted the code optimization and test code as per the
        suggestion. Now the code change is same as the changeset
        e0b025915be8.<br>
        Verified the code reversion with all automated AWT JTreg tests,
        JCK automated tests, reported JCK manual tests and SwingSet2
        test cases.<br>
        <br>
        The test (UnfocusableMaximizedFrameResizablity.java) passes on
        Windows and Ubuntu and fails on Mac, which is expected.<br>
        <br>
        Issue created to fix the original issue: <a
          moz-do-not-send="true"
          href="https://bugs.openjdk.java.net/browse/JDK-8209123">https://bugs.openjdk.java.net/browse/JDK-8209123</a></p>
      <p><br>
        Please review the changes:<br>
        <a moz-do-not-send="true"
          href="http://cr.openjdk.java.net/%7Emhalder/8208125/webrev.01/">http://cr.openjdk.java.net/~mhalder/8208125/webrev.01/</a><br>
        <br>
        Regards,<br>
        Manajit<br>
      </p>
      <br>
      <div class="moz-cite-prefix">On 04/08/18 2:42 AM, Phil Race wrote:<br>
      </div>
      <blockquote type="cite"
        cite="mid:6efd7e84-0157-6989-a220-99a6f3a106b2@oracle.com">
        <meta http-equiv="Content-Type" content="text/html;
          charset=UTF-8">
        So you are saying the fix here cumulatively backs out the fix
        for 7158623 and 8204860<br>
        and reverts the product code to what it was before and
        re-introduces<br>
        7158623 which affected MacOS on all releases of JDK (7 and
        onwards) ?<br>
        I expect that is OK. We can live with a 6 year old bug in 11 as
        much better<br>
        than a stop-ship TCK failure.<br>
        However I'd prefer that you "exactly" reverted the fix<br>
        <br>
        Originally we had<br>
        <pre><span class="removed">-            final boolean resizable = isFrame ? ((Frame)target).isResizable() : (isDialog ? ((Dialog)target).isResizable() : false);

you are proposing to keep the rewrite of that condition in the new fix.
It is much easier for me to be sure of the equivalence if you revert to byte-for-byte what it was before we started.

Ironically, the only other change to this file since you started on this was also backed out !

So once you are done we should be back to the same content as changeset e0b025915be8.

That changeset was also touching maximise so I definitely think we want to make sure JCK passes.
</span></pre>
        <br>
        Have you run the JCK tests on Mac to make sure there are no
        surprises ?<br>
        We don't have a lot of time left here.<br>
        <br>
        Please enumerate all tests you have run on this "reversion" fix.<br>
        I expect it to include all automated jtreg + JCK tests, related
        manual tests, + SwingSet2<br>
        as well as the specific regression test.<br>
        <br>
        Its a bit hard to tell what you are doing with the test, its not
        an exact reversion<br>
        You deleted a lot of machinery and I suppose you want to keep
        that part, just revert<br>
        the parts that verify the functionality you've backed out.<br>
        But you should make sure that if I as I expect the reverted test
        now fails on Mac,<br>
        and since it is automated, this fix should include problem
        listing it on Mac.<br>
        If it does not fail on Mac .. I'd like to know why .. since this
        was the test that originally failed.<br>
        Also you should make sure it definitely still passes on Windows
        + Linux.<br>
        <br>
        And can you file that new bug, with the full back story, and
        reference it here ?<br>
        <br>
        -phil.<br>
        <br>
        <div class="moz-cite-prefix">On 08/03/2018 04:23 AM, Manajit
          Halder wrote:<br>
        </div>
        <blockquote type="cite"
          cite="mid:02694c07-6902-bc4e-4536-3944f941af5c@oracle.com">
          <meta http-equiv="content-type" content="text/html;
            charset=UTF-8">
          Hi All,<br>
          <br>
          Please review the fix for JDK11.<br>
          <br>
          

Bug: <br>
              <a moz-do-not-send="true"
            href="https://bugs.openjdk.java.net/browse/JDK-8208125">
https://bugs.openjdk.java.net/browse/JDK-8208125</a><br>
          <br>
          
Webrev: 
<br>
              <a moz-do-not-send="true"
            href="http://cr.openjdk.java.net/%7Emhalder/8208125/webrev.00/">http://cr.openjdk.java.net/~mhalder/8208125/webrev.00/</a><br>
          <br>
          
Issue: <br>
              This issue and JDK-8208127 was caused due to regression
          caused by fix for the issue JDK-8204860.<br>
          <br>
          Fix:<br>
              I had a relook at the original issue fixed on Windows OS (<a
            moz-do-not-send="true"
            href="https://bugs.openjdk.java.net/browse/JDK-4980161">JDK-4980161</a>)
          and found that the fixes proposed by my earlier fixes were
          wrong w.r.t to the behaviour on Windows and Ubuntu. The issue
          (JDK-4980161) on Windows was that “non-focusable maximized
          frame was resizable” but a focusable maximized frame was
          already non-resizable (working as expected). <br>
          <br>
          Please read the problem description of issue JDK-4980161:
          which says “Typically a maximized frame is not supposed to be
          resizable.”<br>
          <br>
          I verified the behaviour on Windows and Ubuntu and found that
          a maximized Frame
          (frame.setExtendedState(Frame.MAXIMIZED_BOTH)) is not
          resizable irrespective of Frame’s focusable behaviour. On Mac
          OS this behaviour is not implemented i.e. the problem is
          “maximized frame is resizable”.<br>
          <br>
          So the problem which needs to fixed on Mac OS is “Maximized
          frame should not be resizable irrespective of its
          focusability”<br>
          <br>
          Therefor I have reverted my changes to the original state
          (before fix of issue <a moz-do-not-send="true"
            href="https://bugs.openjdk.java.net/browse/JDK-7158623">JDK-7158623</a>)
          and propose to fix the problem on Mac by creating a new issue.<br>
          <br>
          
Regards,<br>
          
Manajit<br>
          <br>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
  </body>
</html>