<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hello, Artem.<div><br></div><div>A couple of comments:</div><div>1. LWWindowPeer: 268 - please remove an empty line.</div><div>2. LWWIndowPeer. isTargetFocusable - the method is not needed at all. </div><div>3. I’m concerned about the test. Do you really need the close button? </div><div>4. frame and window variables are set from main thread and read from EDT. They should be declared volatile.</div><div><br></div><div>Also please run all focus regression and JCK tests, because this area is very sensitive.</div><div><br></div><div>With best regards. Petr.</div><div><br><div><blockquote type="cite"><div>On Jul 22, 2014, at 8:04 PM, artem malinko <<a href="mailto:artem.malinko@oracle.com">artem.malinko@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div>
  

    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  
  <div text="#000000" bgcolor="#FFFFFF">
    Hello, AWT Team.
    <br>
    <br>
    Please review a fix for the issue:
    <br>
    <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8040076">https://bugs.openjdk.java.net/browse/JDK-8047288</a><br>
    The fix is available at:
    <br>
    <a href="http://cr.openjdk.java.net/%7Emcherkas/artem/8047288/webrev.01/">http://cr.openjdk.java.net/~mcherkas/artem/8047288/webrev.01/</a><br>
    <br>
    <span style="font-family: sans-serif; font-size: 13px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: 17.000099182128906px; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; background-color: rgb(240, 240, 240); display: inline !important;">Window.isFocusableWindow() could lead to
      deadlock if it is invoked on AppKit thread. Fix caches result of </span><span style="font-family: sans-serif; font-size: 13px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: 17.000099182128906px; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; background-color: rgb(240, 240, 240); display: inline !important;">Window.isFocusableWindow()
      on a peer level and method is not invoked on AppkitThread.</span><br>
    <br>
    Thank you.
  </div>

</div></blockquote></div><br></div></body></html>