On Thu, 4 Sep 2025 18:24:02 GMT, Alexey Ivanov <[email protected]> wrote:
>> Khalid Boulanouare has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Removes MULTI_FRAME_TEST instance variable and cleans up code
>
> test/jdk/java/awt/Mixing/AWT_Mixing/MixingPanelsResizing.java line 44:
>
>> 42: import test.java.awt.regtesthelpers.Util;
>> 43:
>> 44: /**
>
> I wonder if `frameBorderCounter` needs to run another java process to get the
> result. It should be possible to get the same result by running the code in
> `FrameBorderCounter` class directly in this JVM. Anyway, it's for another
> time.
I have not tried this before, I will check this some time and share my findings.
> test/jdk/java/awt/Mixing/AWT_Mixing/SimpleOverlappingTestBase.java line 70:
>
>> 68: }
>> 69:
>> 70: protected boolean isMultiFramesTest(){
>
> Suggestion:
>
> @Override
> protected boolean isMultiFramesTest(){
This is done in class GlassPaneOverlappingTestBase.java
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2323616834
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2323619931
