RFR: 8307325: Verify the focus owner when non focused windows requesting focus

Damon Nguyen dnguyen at openjdk.org
Thu May 11 23:07:44 UTC 2023


On Thu, 11 May 2023 14:36:04 GMT, lawrence.andrews <duke at 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 52:
> 
>> 50:     private volatile static Button button;
>> 51:     private volatile static TextField textField;
>> 52:     private volatile static int waitTimeout = 1000;
> 
> Add final keyword since value does not change

Agreed. This should be a final constant. And if you make it a constant, I don't think the volatile is necessary.

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

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



More information about the client-libs-dev mailing list