RFR: 8251555: Remove unused focusedWindow field in glass Window to avoid leak
Ambarish Rapte
arapte at openjdk.java.net
Tue Aug 25 10:57:24 UTC 2020
On Sat, 22 Aug 2020 19:39:33 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:
> This is a follow-up to [JDK-8241840](https://bugs.openjdk.java.net/browse/JDK-8241840) that removes an unused static
> `focusedWindow` field and its associated getter and setter from the platform-independent glass Window class. This is
> entirely unused by any of the glass platforms. The Monocle platform keeps track of the `focusedWindow` attribute
> itself. The existing `test.javafx.stage.FocusedWindowNativeTest` automated test, which was added as part of
> [JDK-8241840](https://bugs.openjdk.java.net/browse/JDK-8241840), was failing fairly consistently on one of our test
> machines before this fix, and now passes.
Looks good to me.
-------------
Marked as reviewed by arapte (Reviewer).
PR: https://git.openjdk.java.net/jfx/pull/286
More information about the openjfx-dev
mailing list