RFR: 6726690: SwingUtilities.replaceUI*Map() methods do not remove previously installed maps [v2]

Prasanta Sadhukhan psadhukhan at openjdk.org
Mon Dec 8 13:18:50 UTC 2025


> `SwingUtilities.replaceUIInputMap()` and `SwingUtilities.replaceUIActionMap()` do not actually remove previously installed maps as their Javadoc indicates if `null` is passed as `uiInputMap`/`uiActionMap`
> 
> https://github.com/openjdk/jdk/blob/7e91d34f3e83b4c39d6ce5de34373d7d74d54512/src/java.desktop/share/classes/javax/swing/SwingUtilities.java#L1802-L1803
> https://github.com/openjdk/jdk/blob/7e91d34f3e83b4c39d6ce5de34373d7d74d54512/src/java.desktop/share/classes/javax/swing/SwingUtilities.java#L1827-L1828
> 
> If the passed  `uiInputMap`/`uiActionMap` is null, `JComponent` actually doesn't create a fresh map and returns the previously installed map 
> https://github.com/openjdk/jdk/blob/7e91d34f3e83b4c39d6ce5de34373d7d74d54512/src/java.desktop/share/classes/javax/swing/JComponent.java#L2586-L2595
> which is in contradiction to the `replaceUI*Map` spec so `SwingUtilities `needs to clear the previously installed map which is being done in this fix.

Prasanta Sadhukhan has updated the pull request incrementally with two additional commits since the last revision:

 - Redundant code remove
 - Redundant code remove

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28671/files
  - new: https://git.openjdk.org/jdk/pull/28671/files/63a5ac1c..847b6ba9

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=28671&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=28671&range=00-01

  Stats: 6 lines in 1 file changed: 0 ins; 6 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/28671.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28671/head:pull/28671

PR: https://git.openjdk.org/jdk/pull/28671


More information about the client-libs-dev mailing list