<Swing Dev> RFR: 8255880: UI of Swing components is not redrawn after their internal state changed [v2]
Anton Litvinov
alitvinov at openjdk.java.net
Sun Dec 13 23:56:12 UTC 2020
> Hello colleagues,
>
> Could you please review the fix for the bug specific to Linux OS and Solaris OS, which consists in the fact that, if Swing components are changed through standard public API of these components, while the frame containing these components is in "Frame.ICONIFIED" state or in other words minimized, then, when the frame becomes deiconified the UI of the Swing components does not reflect those changes.
>
> ROOT CAUSE OF THE BUG:
> "javax.swing.RepaintManager.addDirtyRegion0(Container, int, int, int, int)" by design prevents updating regions of containers, when the containers are inside a frame with "Frame.ICONIFIED" state. And at the same time Linux OS specific JDK code does not initiate repaint of the frame, when the frame's peer is notified about change of state from "Frame.ICONIFIED" to other state. More details are available in the bug record.
>
> THE FIX:
> The fix adds code to the method "sun.awt.X11.XFramePeer.handlePropertyNotify(XEvent)" which calls "repaint()" for instance of "XFramePeer", if its state changed from "Frame.ICONIFIED". The fix repeats the approach already existing in macOS specific code, which is in the method "sun.lwawt.LWWindowPeer.notifyIconify(boolean)".
>
> Thank you,
> Anton
Anton Litvinov has updated the pull request incrementally with one additional commit since the last revision:
Removal of required platform restriction from the test
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/1722/files
- new: https://git.openjdk.java.net/jdk/pull/1722/files/c0b5d202..72ebe5ca
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=1722&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=1722&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
Patch: https://git.openjdk.java.net/jdk/pull/1722.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/1722/head:pull/1722
PR: https://git.openjdk.java.net/jdk/pull/1722
More information about the swing-dev
mailing list