On Wed, 7 May 2025 17:11:17 GMT, Martin Fox <m...@openjdk.org> wrote:

>> Thiago Milczarek Sayao has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Fix non-debug build
>
> tests/system/src/test/java/test/robot/javafx/stage/StageOwnershipTest.java 
> line 436:
> 
>> 434:                 stage2::show,
>> 435:                 stage2::close,
>> 436:                 () -> assertTrue(stage1.isFocused()));
> 
> This test looks incorrect. If closing an app modal window should focus the 
> parent (and I think it should) then stage0 should be focused.

But `stage1` is the owner of `stage2`

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1789#discussion_r2078282466

Reply via email to