<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Hi, Alexander.<br>
      A few notes:<br>
       - _peer.setMaximizedBounds(_peer.getMaximizedBounds()); this line
      looks odd, probably we can call updateMaximizedBounds in
      LWWindowPeer.
      <meta http-equiv="content-type" content="text/html;
        charset=windows-1252">
      initializeImpl()?<br>
       - Do we need to call maximize(); in the initialize() or we can
      leave it in the setVisible()? It seems that after we set
      NSWindowStandardFrame isZoomed should work properly in the
      callback?<br>
       - Why we cannot return DefaultMaximizedBounds from the
      getMaximizedBounds if it was not set? Then it will not be
      necessary to check it to null everywhere.<br>
       - Can we call updateMaximizedBounds(); in
      LWWindowPeer.displayChanged() only in case of new device?<br>
      <br>
      On 14.05.15 12:40, Alexander Scherbatiy wrote:<br>
    </div>
    <blockquote cite="mid:55546D89.7080704@oracle.com" type="cite">On
      5/12/2015 4:44 PM, Alexander Scherbatiy wrote: <br>
      <blockquote type="cite"> <br>
        Could you review the updated fix: <br>
        - maximized bounds are updated after switching to new screen if
        they are not set by a user <br>
        - the test for the maximized window moved from one screen to
        another is added <br>
      </blockquote>
         Sorry. Just forgot to add the webrev link: <br>
           <a class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Ealexsch/8065739/webrev.04/">http://cr.openjdk.java.net/~alexsch/8065739/webrev.04/</a>
      <br>
      <br>
         Thanks, <br>
         Alexandr. <br>
      <br>
      <blockquote type="cite"> <br>
        <br>
        Thanks, <br>
        Alexandr. <br>
        <br>
        On 5/8/2015 3:42 PM, Sergey Bylokhov wrote: <br>
        <blockquote type="cite">Hi, Alexander. <br>
          Does the new tests cover all corner cases which we discussed
          offline? <br>
          <br>
          On 08.05.15 15:28, Alexander Scherbatiy wrote: <br>
          <blockquote type="cite"> <br>
              Hello, <br>
            <br>
             Could you review the updated fix: <br>
               <a class="moz-txt-link-freetext"
              href="http://cr.openjdk.java.net/%7Ealexsch/8065739/webrev.03">http://cr.openjdk.java.net/~alexsch/8065739/webrev.03</a>
            <br>
            <br>
             - The -windowWillUseStandardFrame:defaultFrame: method is
            implemented in AWTWindow to correctly get and check a size
            of the maximized window. <br>
               After that the -isZoomed method correctly defines the
            zoomed state for both visible and not visible window. <br>
            <br>
             Thanks, <br>
             Alexandr. <br>
            <br>
            On 4/29/2015 7:03 PM, Alexander Scherbatiy wrote: <br>
            <blockquote type="cite"> <br>
               Hello, <br>
              <br>
               Could you review the updated fix: <br>
                 <a class="moz-txt-link-freetext"
                href="http://cr.openjdk.java.net/%7Ealexsch/8065739/webrev.02">http://cr.openjdk.java.net/~alexsch/8065739/webrev.02</a>
              <br>
              <br>
                - the zoom state is applied after NSWindow creation if
              Frame has MAXIMIZED_BOTH state. <br>
              <br>
               Thanks, <br>
               Alexandr. <br>
              <br>
              On 4/15/2015 10:57 PM, Sergey Bylokhov wrote: <br>
              <blockquote type="cite">Hi, Alexander. <br>
                As far as I understand the main issue is that we get a
                native notifications during window initialization(after
                nswindows was created and before zoom method will be
                called)? Probably we can apply the zoom state when we
                create a window in one appkit call? <br>
                <br>
                On 08.04.15 17:54, Alexander Scherbatiy wrote: <br>
                <blockquote type="cite">Hello, <br>
                  <br>
                  Could you review the fix: <br>
                    bug: <a class="moz-txt-link-freetext"
                    href="https://bugs.openjdk.java.net/browse/JDK-8065739">https://bugs.openjdk.java.net/browse/JDK-8065739</a>
                  <br>
                    webrev: <a class="moz-txt-link-freetext"
                    href="http://cr.openjdk.java.net/%7Ealexsch/8065739/webrev.00">http://cr.openjdk.java.net/~alexsch/8065739/webrev.00</a>
                  <br>
                  <br>
                    Use case: <br>
                    ------------ <br>
                    Frame frame = new Frame(); <br>
                    frame.pack(); <br>
                    frame.setBounds(screenBounds); <br>
                    frame.setVisible(true); <br>
                    ------------ <br>
                  <br>
                    The frame is in the normal state but Mac OS X treats
                  a frame with screen bounds as zoomed. <br>
                    The setVisible() method calls unmaximize() method to
                  return the zoomed window to the normal state. <br>
                    The native zoom() method sets the initial sizes for
                  the window which are zero because of the pack() call.
                  <br>
                  <br>
                  <br>
                    There is already checkZoom() method which
                  synchronizes native window zoom state and and the
                  frame extended state when a frame is resized by mouse.
                  <br>
                    However it can't be always used for the
                  synchronization between native window zoom state and
                  setBounds() call because of the following case: <br>
                  <br>
                    ----------------- <br>
                    Frame frame = new Frame(); <br>
                    frame.setBounds(smallBounds); <br>
                    frame.setExtendedState(Frame.MAXIMIZED_BOTH); <br>
                    frame.setVisible(true); <br>
                    ----------------- <br>
                  <br>
                    This is the valid scenario where the frame should be
                  maximized. <br>
                  <br>
                    The solution synchronizes the zoom and the frame
                  state only when the native window is zoomed. <br>
                    In opposite case we need to return from the zoomed
                  sizes to previous sizes which are the same as for the
                  zoomed window. <br>
                  <br>
                  Thanks, <br>
                  Alexandr. <br>
                  <br>
                </blockquote>
                <br>
                <br>
              </blockquote>
              <br>
            </blockquote>
            <br>
          </blockquote>
          <br>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>