> Please review the following fix for jdk16. > > Issue : There is a JToggleButton that will post/take down a JPopupMenu when > the button is selected. If the button is selected and the menu is not posted > the action listener will post the menu. If the button is selected and the > menu is displayed the action listener will take the menu down. The use case > is: > 1 - select button > 2 - menu posted > 3 - select button > 4 - menu taken down > > With MetalLookAndFeel the above use case works fine, but with > WindowsLookAndFeel the second button selection does not fire the > actionPerformed event, button needs to be selected third time for the menu > to be taken down. > > The issue is that the button must be selected twice after the menu is posted > to have the actionPerformed event to fire when using the Windows look and > feel. > > Fix : MouseGrabber is not removed while uninstalling the listeners in the > BasicPopupMenuUI. > By removing the mouseGrabber in the uninstallListeners() methods fixes this > issue. > > Added a test to test the same in all the LookAndFeels
Tejpal Rebari has updated the pull request incrementally with one additional commit since the last revision: modified the test after suggestions ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/600/files - new: https://git.openjdk.java.net/jdk/pull/600/files/a60ffa67..65c08eda Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=600&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=600&range=00-01 Stats: 10 lines in 1 file changed: 7 ins; 1 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/600.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/600/head:pull/600 PR: https://git.openjdk.java.net/jdk/pull/600
