On Thu, 4 May 2023 05:41:43 GMT, Ravi Gupta <d...@openjdk.org> wrote:

> Write a test Check, when the top level Window is not the focused Window 
> requesting for focus and becoming the Focus Owner for any Component in that 
> Window checking the following
> 
> 1.The Component is the Focus Owner and the Window becomes the focused Window 
> If the platform supports cross requesting focus
> across Windows.
> 2.The request is remembered and  be granted when the Window is later focused 
> If the platform does not support requesting focus
> across Windows.
> 
> Testing:
> Tested using Mach5(20 times per platform) in macos,linux and windows and got 
> all pass.

test/jdk/java/awt/Focus/CrossFocusRequestTest/CrossFocusRequestTest.java line 
85:

> 83:             }
> 84: 
> 85:             robot.waitForIdle();

Did you mean to put delays here instead of waitForIdles? I believe you already 
have setAutoWaitForIdle to handle this. Might be better to set a delay(1000) 
instead for stability.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13790#discussion_r1191763671

Reply via email to