RFR: 8158801: [TEST_BUG] Mixing tests fail because of focus workaround trick [v37]

Khalid Boulanouare duke at openjdk.org
Thu Sep 4 15:49:47 UTC 2025


On Thu, 4 Sep 2025 14:12:50 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

>> Khalid Boulanouare has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Organizes imports
>
> test/jdk/java/awt/Mixing/AWT_Mixing/GlassPaneOverlappingTestBase.java line 28:
> 
>> 26: import java.awt.event.MouseAdapter;
>> 27: import java.awt.event.MouseEvent;
>> 28: import java.lang.Override;
> 
> All the classes from `java.lang.*` are imported implicitly.
> 
> That is, remove `import java.lang.Override`.

lang package imports removed.

> test/jdk/java/awt/Mixing/AWT_Mixing/GlassPaneOverlappingTestBase.java line 100:
> 
>> 98:     protected boolean isMultiFramesTest(){
>> 99:         return false;
>> 100:     }
> 
> There should be no blank line before the closing brace of a method or constructor. That is the snippet should like this:
> 
> 
>     public GlassPaneOverlappingTestBase(boolean defaultClickValidation) {
>         super(defaultClickValidation);
>     }
> 
>     @Override
>     protected boolean isMultiFramesTest() {
>         return false;
>     }

Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2322618892
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2322627072


More information about the client-libs-dev mailing list