On Thu, 4 Sep 2025 12:24:29 GMT, Khalid Boulanouare <[email protected]> wrote:

>> The above approach may require additional refactoring, and I see that class 
>> initialisers are used to set values of flags.
>
> I suggest the following changes : 
> 
> 
>  public GlassPaneOverlappingTestBase() {
>         this(true);
>     }
> 
>     public GlassPaneOverlappingTestBase(boolean defaultClickValidation) {
>         super(defaultClickValidation);
>         multiFramesTest =false;
>     }

The previous suggestion will not work, please ignore it.

I am looking at creating a method to set the value of MULTI_FRAMES_TEST, this 
will allow us to do the changes only in the 2 parent classes. I will try this 
and confirm if the tests pass.

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

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

Reply via email to