On Fri, 1 Aug 2025 06:08:12 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> test/jdk/java/awt/Frame/MiscUndecorated/ActiveAWTWindowTest.java line 59:
>> 
>>> 57:     private static final CountDownLatch windowActivatedLatch = new 
>>> CountDownLatch(1);
>>> 58:     private static final CountDownLatch windowDeactivatedLatch = new 
>>> CountDownLatch(1);
>>> 59:     private static final CountDownLatch windowFocusGainedLatch = new 
>>> CountDownLatch(1);
>> 
>> I see where we decrement this latch but i do not see where we check it or 
>> wait on it. Is it really needed?
>
> also no need to use ExtendedRobot as all I can see is mousePress/mouseRelease 
> which can be used from java.awt.Robot class itself..

static final variables can be capitalized.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26471#discussion_r2247144121

Reply via email to