JDK-8002114 : fix failed for JDK-7160951: [macosx] ActionListener called twice for JMenuItem using ScreenMenuBar
  • Type: Bug
  • Component: client-libs
  • Sub-Component: javax.swing
  • Affected Version: 8
  • Priority: P2
  • Status: Closed
  • Resolution: Fixed
  • OS: os_x
  • CPU: generic
  • Submitted: 2012-11-01
  • Updated: 2016-03-31
  • Resolved: 2012-11-02
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
7u10Fixed 8 b64Fixed
Related Reports
Duplicate :  
Relates :  
Relates :  
Relates :  
Relates :  
Relates :  
ActionListener called once when run code from JDK-7160951 but if change accellerator to Ctrl-E, for example, then listener still called twice 

test results:
failed with jdk8b57, jdk8b58, jdk7u10b12, jdk7u6b06
passed with jdk7u6b05, jdk8b54

see comments for more details
JDK-8003901 was filed to provide a regression test for this issue, and as a result additional test case was added to javax/swing/JMenuItem/ActionListenerCalledTwice/ActionListenerCalledTwiceTest.java

SQE is OK to take this.

this problem is reproducible with jdk7u10b14 comment corrected.

JDK-7160951(JDK-8002114) and JDK-7186371 should be fixed simultaneously, otherwise we introduce new regression in 7u10.

recheck shows that test passed with jdk7u9b05 and failed since jdk7u10b08

Seems to be relatively corner case use case as Control is not main accelerator button on Mac OS. SQE recommends considering to postpone to 7u12 as impact is low and general guidance is to reduce risk to the release.

Need SQE to weigh in

The fix is a rather simple and low risk, just one changed line. I suggest we push it into 7u10.