On Tue, 9 Sep 2025 18:40:31 GMT, Phil Race <p...@openjdk.org> wrote:
>> test/jdk/java/awt/Focus/FrameInitialFocusTest/FrameInitialFocusTest.java
>> line 66:
>>
>>> 64: Util.waitForIdle(robot);
>>> 65: if (!button.hasFocus()) {
>>> 66: throw new RuntimeException("Appletviewer
>>> doesn't set default focus correctly.");
>>
>> AppletViewer => Frame ?
>
> The frame isn't setting the focus. "Focus not set on button" might be better
> , but I do agree the word "Appletviewer" needs to be gone !
>
> And I was suggesting to just make the test name "IntitialFocusTest"
> I thought about changing it to Frame IntitialFocusTest, but it isn't the
> Frame we expect to get focus.
> It's the button.
Updated
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27136#discussion_r2335301390
