RFR: 6850365: java/awt/Focus/ActivateFocusTest.java fails [v2]

Prasanta Sadhukhan psadhukhan at openjdk.org
Tue Sep 16 02:58:54 UTC 2025


On Mon, 15 Sep 2025 17:13:32 GMT, Phil Race <prr at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Check for failure in 1st frame activation
>
> test/jdk/java/awt/Focus/ActivateFocusTest.java line 57:
> 
>> 55:             boolean testFailed = false;
>> 56:             for (int i = 0; i < NUM_FRAMES; i++) {
>> 57:                 testFailed = (af[i].lw.focusCounter > 1);
> 
> I don't understand this code. If the first one fails, and the second one passes it will pass the test when I presume it should have failed it.

I have updated the PR to check for 1st frame `testFailed ` too

Also, regarding renaming the test to FocusActiveFocusTest.java, I guess there's too many Focus in the name and I dont see any wrong in the present name ActivateFocus which seems to be right considering what it is testing, which is `focus on window activation` so unless it's a deal-breaker, I guess we can keep the name

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27214#discussion_r2350536351


More information about the client-libs-dev mailing list