<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Hi Sergey,<br>
    <br>
    Right. I introduced an exit condition:
    <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ssadetsky/8085895/webrev.01/">http://cr.openjdk.java.net/~ssadetsky/8085895/webrev.01/</a><br>
    <br>
    --Semyon<br>
    <br>
    <div class="moz-cite-prefix">On 6/9/2015 9:13 PM, Sergey Bylokhov
      wrote:<br>
    </div>
    <blockquote cite="mid:55772CB2.9030707@oracle.com" type="cite">
      <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
      <div class="moz-cite-prefix">Hi, Semyon.<br>
        The fix looks fine, but I worry about possible infinite loops, 
        since we have a loop here peer->componet->peer. Please
        check the next use case:<br>
        TextField.setBackground()-><b>peer.setBackground()</b>->JTextComponent.setBackground()-><b>
          peer.setBackground()</b><br>
        Will it works?<br>
        <br>
        On 09.06.15 19:43, Semyon Sadetsky wrote:<br>
      </div>
      <blockquote cite="mid:557717AA.7010200@oracle.com" type="cite">Hello,

        <br>
        <br>
        Please review fix for JDK9: <br>
        <br>
        bug: <a moz-do-not-send="true" class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8085895">https://bugs.openjdk.java.net/browse/JDK-8085895</a>
        <br>
        webrev: <a moz-do-not-send="true" class="moz-txt-link-freetext"
href="http://cr.openjdk.java.net/%7Essadetsky/8085895/webrev.00/">http://cr.openjdk.java.net/~ssadetsky/8085895/webrev.00/</a>
        <br>
        <br>
        In 8074028 overridden getPeer() was simply removed in XWindows
        peers of TextField and TextArea but that was not enough to
        preserve the components functionality. <br>
        In the new design Component.getPeer() doesn't exist anymore. To
        restore the previous logic completely it requires to override in
        the peer delegate component all the Component methods those used
        getPeer() before. But I think that would be a bad piece of code.
        I propose more short fix that was tested to be working using
        jtreg TextFiled/* and TextArea/* tests. <br>
        <br>
        --Semyon <br>
        <br>
      </blockquote>
      <br>
      <br>
      <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
    </blockquote>
    <br>
  </body>
</html>