<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">10.09.2015 19:26, Semyon Sadetsky
      пишет:<br>
    </div>
    <blockquote cite="mid:55F1AF2A.4020900@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      I have added size assignment to the target component. Please
      review.<br>
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Essadetsky/8011616/webrev.01/">http://cr.openjdk.java.net/~ssadetsky/8011616/webrev.01/</a><br>
    </blockquote>
    <br>
    Probably we can get an accessor only once? this makes code a little
    bit compact and clear?<br>
    <br>
    <blockquote cite="mid:55F1AF2A.4020900@oracle.com" type="cite"> <br>
      --Semyon<br>
      <br>
      <br>
      <div class="moz-cite-prefix">On 7/31/2015 9:28 PM, Semyon Sadetsky
        wrote:<br>
      </div>
      <blockquote cite="mid:55BBBE5A.1020102@oracle.com" type="cite">
        <meta content="text/html; charset=utf-8"
          http-equiv="Content-Type">
        You are right in windows size is limited to 65535 and I tested
        smaller value. <br>
        In Linux WM will limit window size to entire of the screen. I
        will add size assignment as well. <br>
        <br>
        --Semyon<br>
        <br>
        <div class="moz-cite-prefix">On 7/31/2015 7:12 PM, Sergey
          Bylokhov wrote:<br>
        </div>
        <blockquote cite="mid:55BB9E79.8040800@oracle.com" type="cite">
          <meta content="text/html; charset=utf-8"
            http-equiv="Content-Type">
          <div class="moz-cite-prefix">On 31.07.15 18:00, Semyon
            Sadetsky wrote:<br>
          </div>
          <blockquote cite="mid:55BB8D7D.70905@oracle.com" type="cite">
            <meta content="text/html; charset=utf-8"
              http-equiv="Content-Type">
            <br>
            <blockquote cite="mid:55BB8897.9090305@oracle.com"
              type="cite"> On windows is changed in awt_Window.cpp.
              WmMove<br>
            </blockquote>
            <br>
            What I see there:<br>
            <br>
            <meta http-equiv="content-type" content="text/html;
              charset=utf-8">
            (env)->SetIntField(target, AwtComponent::xID, rect.left);<br>
                (env)->SetIntField(target, AwtComponent::yID,
            rect.top);<br>
            <br>
            Are you talking about size or screen position? <br>
          </blockquote>
          Sorry the method name is AwtWIndow::WmSize in awt_Window.cpp,
          where we update the target and then post the event.<br>
          <br>
          <blockquote cite="mid:55BB8D7D.70905@oracle.com" type="cite">
            The initial window size is preserved on Windows. On OSX it
            can be different if
            <meta http-equiv="content-type" content="text/html;
              charset=utf-8">
            notifyReshape() is called upon window create.<br>
          </blockquote>
          The window size can be change externally by the different
          reason, so we should always relies on the value which was
          reported by the system.<br>
          <blockquote cite="mid:55BB8D7D.70905@oracle.com" type="cite">
            I don't see any extra issues related to the size currently. 
            <br>
            <br>
            --Semyon<br>
            <meta http-equiv="content-type" content="text/html;
              charset=utf-8">
            <br>
            <blockquote cite="mid:55BB8897.9090305@oracle.com"
              type="cite"> In general window manager can set a different
              size/location than we try to set, so we should take a
              correct values in the callback and update the target
              component, and after that post event that the bounds was
              changed.<br>
              <br>
              <blockquote cite="mid:55BB83F0.4050505@oracle.com"
                type="cite"> - the utility method was introduced because
                decorated window can be moved and it is utilized from
                several places. It is not the case for undecorated
                window. <br>
              </blockquote>
              It can be moved by the window manager.<br>
              <blockquote cite="mid:55BB83F0.4050505@oracle.com"
                type="cite"> Not sure that container screen position
                makes any sense for its content size event. Is it
                specified somewhere? Or can you provide an example when
                it is necessary?<br>
              </blockquote>
              I am not sure. It needs to be checked.<br>
              <blockquote cite="mid:55BB83F0.4050505@oracle.com"
                type="cite"> <br>
                --Semyon<br>
                <br>
                <div class="moz-cite-prefix">On 7/31/2015 4:14 PM,
                  Sergey Bylokhov wrote:<br>
                </div>
                <blockquote cite="mid:55BB74AA.2040407@oracle.com"
                  type="cite">
                  <meta content="text/html; charset=utf-8"
                    http-equiv="Content-Type">
                  <div class="moz-cite-prefix">Hi, Semyon.<br>
                    A few questions.<br>
                     - Why only location is fixed, an update of size if
                    not necessary(if for some reason the size was
                    changed by the system like location in this case)?<br>
                     - Note that we should update the target state
                    before we post an event that the size is changed(we
                    post them in XWindow.handleConfigureNotifyEvent). It
                    seems that
                    <meta http-equiv="content-type" content="text/html;
                      charset=utf-8">
                    some of the peers has utility methods for this(like
                    XDecoratedPeer.handleMoved).<br>
                    <br>
                    On 31.07.15 11:52, Semyon Sadetsky wrote:<br>
                  </div>
                  <blockquote cite="mid:55BB3762.1070102@oracle.com"
                    type="cite">Hello, <br>
                    <br>
                    Please review fix for JDK9: <br>
                    bug: <a moz-do-not-send="true"
                      class="moz-txt-link-freetext"
                      href="https://bugs.openjdk.java.net/browse/JDK-8011616">https://bugs.openjdk.java.net/browse/JDK-8011616</a>
                    <br>
                    webrev: <a moz-do-not-send="true"
                      class="moz-txt-link-freetext"
                      href="http://cr.openjdk.java.net/%7Essadetsky/8011616/webrev.00/">http://cr.openjdk.java.net/~ssadetsky/8011616/webrev.00/</a>
                    <br>
                    <br>
                    WM sends the real window position in the
                    configuration event but window peer does not set it
                    to the target. Solution is: do set. <br>
                    <br>
                    --Semyon <br>
                  </blockquote>
                  <br>
                  <br>
                  <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
                </blockquote>
                <br>
              </blockquote>
              <br>
              <br>
              <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
            </blockquote>
            <br>
          </blockquote>
          <br>
          <br>
          <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>