<AWT Dev> Review request for 8047288: [macosx] Endless loop in EDT on Mac

artem malinko artem.malinko at oracle.com
Wed Jul 23 14:55:51 UTC 2014

Hi, Petr. I ran focus regression tests and jck tests on awt. For fixed 
jdk results is the same. Except my new test, of course which is not 
passed on not fixed jdk:) And also I fixed issues in test. New webrev:

On 7/22/2014 8:23 PM, Petr Pchelko wrote:
> Hello, Artem.
> A couple of comments:
> 1. LWWindowPeer: 268 - please remove an empty line.
> 2. LWWIndowPeer. isTargetFocusable - the method is not needed at all.
> 3. I’m concerned about the test. Do you really need the close button?
> 4. frame and window variables are set from main thread and read from 
> EDT. They should be declared volatile.
> Also please run all focus regression and JCK tests, because this area 
> is very sensitive.
> With best regards. Petr.
>> On Jul 22, 2014, at 8:04 PM, artem malinko <artem.malinko at oracle.com 
>> <mailto:artem.malinko at oracle.com>> wrote:
>> Hello, AWT Team.
>> Please review a fix for the issue:
>> https://bugs.openjdk.java.net/browse/JDK-8047288
>> The fix is available at:
>> http://cr.openjdk.java.net/~mcherkas/artem/8047288/webrev.01/ 
>> <http://cr.openjdk.java.net/%7Emcherkas/artem/8047288/webrev.01/>
>> Window.isFocusableWindow() could lead to deadlock if it is invoked on 
>> AppKit thread. Fix caches result of Window.isFocusableWindow() on a 
>> peer level and method is not invoked on AppkitThread.
>> Thank you.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20140723/cb66d6c1/attachment-0001.html>

More information about the awt-dev mailing list