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

[Jose Pereda]

On Sat, 22 Apr 2023 18:47:54 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 incrementally with one additional commit since the last revision:
> 
>   Change call to requestFocus

Tested successfully on Android and Linux. I've added a few comments.

modules/javafx.graphics/src/main/java/com/sun/glass/ui/gtk/GtkWindow.java line 51:

> 49:     // empty - not needed by this implementation
> 50:     @Override
> 51:     protected void _updateViewSize(long ptr) {}

Remove the comment?

modules/javafx.graphics/src/main/java/com/sun/glass/ui/gtk/GtkWindow.java line 51:

> 49:     // empty - not needed by this implementation
> 50:     @Override
> 51:     protected native void _updateViewSize(long ptr);

Is there a system test that can be used to verify these changes on Linux? Since you have mentioned this is the same as #1054, I take that SetSceneScalingTest can be used. 

Also, I'm not sure how you would run WinIconified test attached to the JBS ticket to see that it fails or passes. If that can be converted to system test, could you add it to this PR?

modules/javafx.graphics/src/main/java/com/sun/glass/ui/gtk/GtkWindow.java line 51:

> 49:     // empty - not needed by this implementation
> 50:     @Override
> 51:     protected void _updateViewSize(long ptr) {}

Fix copyright header of modified files

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

PR Review: https://git.openjdk.org/jfx/pull/1110#pullrequestreview-1397890332
PR Review Comment: https://git.openjdk.org/jfx/pull/1110#discussion_r1175221410
PR Review Comment: https://git.openjdk.org/jfx/pull/1110#discussion_r1175425902
PR Review Comment: https://git.openjdk.org/jfx/pull/1110#discussion_r1175428836