Bug ID: JDK-8002114 fix failed for JDK-7160951: [macosx] ActionListener called twice for JMenuItem using ScreenMenuBar

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

Versions (Unresolved/Resolved/Fixed)

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.

JDK 7	JDK 8
7u10Fixed	8 b64Fixed

Related Reports

Duplicate :	JDK-8024328 - [macosx] ActionListeners fire twice for JMenuItems still in the ScreenMenuBar.
Relates :	JDK-8152492 - [macosx swing] double key event actions when using Mac menubar
Relates :	JDK-7186371 - [macosx] Main menu shortcuts not displayed (7u6 regression)
Relates :	JDK-7160951 - [macosx] ActionListener called twice for JMenuItem using ScreenMenuBar
Relates :	JDK-8003901 - [macosx] Need test for JDK-8002114
Relates :	JDK-8024387 - Provide a regression test for JDK-8002114

Description

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

Comments

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
16-09-2013
SQE is OK to take this.
14-11-2012
this problem is reproducible with jdk7u10b14 comment corrected.
12-11-2012
JDK-7160951(JDK-8002114) and JDK-7186371 should be fixed simultaneously, otherwise we introduce new regression in 7u10.
07-11-2012
recheck shows that test passed with jdk7u9b05 and failed since jdk7u10b08
07-11-2012
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.
07-11-2012
Need SQE to weigh in
05-11-2012
The fix is a rather simple and low risk, just one changed line. I suggest we push it into 7u10.
01-11-2012