[jfx17] RFR: 8240640: [macos] Wrong focus behaviour with multiple Alerts [v3]

[Ambarish Rapte]

On Fri, 23 Jul 2021 17:40:32 GMT, Pankaj Bansal <pbansal at openjdk.org> wrote:

>> The bug is a regression as a result of fix done for JDK-8227366 and is reproducible on Linux and Mac. This fix is being reverted in this change and a new bug (JDK-8271054) has been created to redo the JDK-8227366
>> 
>> An automated testcase is being added to make sure similar regression is not introduced when working on the redo bug. The automated testcase fails without the current change and passes after the change.
>> 
>> The testcase works fine on Mac and Linux, but I see some issues in windows. The stage is minimised after calling stage.show without calling stage.setAlwaysOnTop(true). I see that there are other tests like DualWindowsTest.java which are failing for same reason. Also, if I remove the stage.setAlwaysOnTop(true) from tests like FocusParentWindowOnChildCloseTest.java,  the stage remains minimised there as well. This seems wrong as calling stage.setAlwaysOnTop(true) should not be required to bring stage window in focus on running the test. This may be an issue on my Window machine or something similar. I hope reviewers can shed some light on this.
>
> Pankaj Bansal has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix formatting for binary operators

looks good to me.

Marked as reviewed by arapte (Reviewer).

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

PR: https://git.openjdk.java.net/jfx/pull/581