On Mon, 15 Jun 2026 00:55:17 GMT, Thiago Milczarek Sayao <[email protected]>
wrote:
>> modules/javafx.graphics/src/main/native-glass/gtk/glass_window.cpp line 1680:
>>
>>> 1678: }
>>> 1679:
>>> 1680: if (mapped) {
>>
>> This looks a bit suspicious: `move_resize()` uses `mapped` to decide whether
>> to call `gtk_window_resize()` or `gtk_window_set_default_size()`. The old
>> code used `gtk_widget_get_realized()` and resized any realized window. Since
>> the `mapped` field is always false for a `POPUP` window, the new code might
>> not resize an already-realized popup.
>
> Popups go through the same process of setting `mapped` to `true`. However,
> I'll do some testing to verify popup move and resize behavior.
Only `WindowContext::process_map()` seems to set `mapped = true`, but not if
`window_type == POPUP`.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2139#discussion_r3410689214
