RFR: 8288893: Popup and its subclasses cannot input text from InputMethod [v3]

[Kevin Rushforth]

On Wed, 4 Dec 2024 15:36:00 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> In a PopupWindow the scene there is no API to set the scene. It's managed internally by the system and should not change over the lifetime of the popup window. I will take a second look at that code next week to verify.
>
> One can extend `PopupWindow` and call the protected `setScene()`, so this is a valid scenario, I think.

PopupWindow already specifies that subclasses should not do this:


Note to subclasses: the scene used by PopupWindow is very specifically managed by PopupWindow. This method is overridden to throw UnsupportedOperationException. You cannot specify your own scene.


So no, it is not a valid use case.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1634#discussion_r1870055363