RFR: 8306121: Scene not rendered initially when changing scenes after fix for JDK-8296621 [v6]

Lukasz Kostyra lkostyra at openjdk.org
Mon May 1 12:44:26 UTC 2023


On Mon, 24 Apr 2023 23:35:14 GMT, Thiago Milczarek Sayao <tsayao at openjdk.org> wrote:

>> I replicated the fix on #1054 on Linux.
>> 
>> Also fixes [Monocle: Stage no longer gets focus after fix for JDK-8296621](https://bugs.openjdk.org/browse/JDK-8304734)
>
> Thiago Milczarek Sayao has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 46 additional commits since the last revision:
> 
>  - Review changes
>  - Merge branch 'master' into change_scene_fix
>  - Change call to requestFocus
>  - Revert "The window could start iconified."
>    
>    This reverts commit f92e25cbcc24fd772b9d6bb25a92386ec0561313.
>  - The window could start iconified.
>  - Request Focus on setVisible
>  - fix mistake
>  - Scene not rendered initially when changing scenes after fix for JDK-8296621
>  - Merge branch 'openjdk:master' into master
>  - Merge branch 'openjdk:master' into master
>  - ... and 36 more: https://git.openjdk.org/jfx/compare/4733565a...2649d15c

I verified the changes, in general the `testSecondSetScene` test is now passing on all platforms. However, on Windows and macOS I observed two tests from controls test suite failing:
* MouseEventFirerTest. testAnchorRight[0]
* ComboBoxTest. test_rt28245

When I change branches to before all these changes - 48b5fea374 - or even before the most recent master that's merged to these tests - 559ad1139b - these tests pass. Seems like these two failures were introduced by this PR.

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

PR Review: https://git.openjdk.org/jfx/pull/1110#pullrequestreview-1407560199


More information about the openjfx-dev mailing list