Integrated: 8353314: macOS: Inconsistent fullscreen behavior

Martin Fox mfox at openjdk.org
Fri May 2 02:31:49 UTC 2025


On Wed, 30 Apr 2025 17:52:01 GMT, Martin Fox <mfox at openjdk.org> wrote:

> macOS will allow any window to enter fullscreen mode but won't expand the window's size if the resizable bit isn't set in the window's style mask. For undecorated stages the code has to set this bit before entering fullscreen and restore the old value after exiting fullscreen.
> 
> The old code was taking a pointer to an NSWindow and casting it to a pointer to an unrelated type (GlassWindow). It was also making an unnecessary check. windowWillEnterFullScreen stashes away the state of the resizable bit before setting it and windowDidExitFullScreen restores the old state. There's no need for setResizableForFullscreen to check anything, it just needs to do what it's told.
> 
> System tests for this case are underway as part of PR #1789.
> 
> You might have difficulty reproducing the bug in the master branch.  The old code was doing a bogus pointer cast and then dereferencing it to check a state flag so the code sometimes worked and sometimes didn't.

This pull request has now been integrated.

Changeset: 096765a0
Author:    Martin Fox <mfox at openjdk.org>
URL:       https://git.openjdk.org/jfx/commit/096765a0e51c5cf67d64b4b5f38ef3da3914db35
Stats:     9 lines in 1 file changed: 1 ins; 3 del; 5 mod

8353314: macOS: Inconsistent fullscreen behavior

Reviewed-by: angorya, kcr

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

PR: https://git.openjdk.org/jfx/pull/1799


More information about the openjfx-dev mailing list