FULL PRODUCT VERSION :
java version "1.6.0-beta2"
Java(TM) SE Runtime Environment (build 1.6.0-beta2-b84)
Java HotSpot(TM) Client VM (build 1.6.0-beta2-b84, mixed mode, sharing)

java version "1.5.0_06"
Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_06-b05)
Java HotSpot(TM) Client VM (build 1.5.0_06-b05, mixed mode, sharing)


ADDITIONAL OS VERSION INFORMATION :
Microsoft Windows XP [Version 5.1.2600]


A DESCRIPTION OF THE PROBLEM :
The setSelectedIcon() method for a JMenuItem and JCheckBoxMenuItem has no
effect. Regardless of whether or not the item is selected, it will be drawn using the
unselected icon.


STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
1) Run the test program.
2) Observe the colors of the Xs in the menu and the toolbar buttons.
3) Choose each menu item, and again observe the colors of the Xs.
4) Try clicking the tool bar buttons and observe the colors.


EXPECTED VERSUS ACTUAL BEHAVIOR :
EXPECTED -
The color in the menu item should always match that in the corresponding
toolbar button.

ACTUAL -
For the first two menu items, the color in the menu is always gray, even
when the toolbar button is red.


REPRODUCIBILITY :
This bug can be reproduced always.

---------- BEGIN SOURCE ----------
import java.awt.*;
import javax.swing.*;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;

/** @noinspection HardCodedStringLiteral*/
@SuppressWarnings({"MagicNumber", "DuplicateStringLiteralInspection"})
public class CheckBoxMenuItemBug extends JPanel {
private static JFrame sMainFrame;

public static void main(String[] args) {
makeFrame();
}

private static void makeFrame() {
sMainFrame = new JFrame("CheckBoxMenuItemBug"); // NON-NLS
sMainFrame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
sMainFrame.getContentPane().add(new CheckBoxMenuItemBug(), BorderLayout.CENTER);
sMainFrame.setBounds(10, 10, 600, 325);
sMainFrame.setVisible(true);
}

public CheckBoxMenuItemBug() {
super(new BorderLayout());
JToolBar tBar = new JToolBar();
add(tBar, BorderLayout.NORTH);
final Icon unselectedIcon = new UnselectedIcon();
final Icon selectedIcon = new SelectedIcon();
JMenuBar mBar = new JMenuBar();

JMenu testMenu = new JMenu("Test");

JMenuItem itemOne = new JMenuItem("JMenuItem", unselectedIcon);
itemOne.setModel(new JToggleButton.ToggleButtonModel());
itemOne.setSelectedIcon(selectedIcon);
testMenu.add(itemOne);
tBar.add(createToolBarButton(itemOne));

JCheckBoxMenuItem itemTwo = new JCheckBoxMenuItem("JCheckBoxMenuItem", unselectedIcon);
itemTwo.setSelectedIcon(selectedIcon);
testMenu.add(itemTwo);
tBar.add(createToolBarButton(itemTwo));

JMenuItem itemThree = new JMenuItem("Workaround One", unselectedIcon);
itemThree.setModel(new JToggleButton.ToggleButtonModel());
itemThree.setSelectedIcon(selectedIcon);
testMenu.add(itemThree);
tBar.add(createToolBarButton(itemThree));
workaround(itemThree, selectedIcon, unselectedIcon);

JCheckBoxMenuItem itemFour = new JCheckBoxMenuItem("Workaround Two", unselectedIcon);
itemFour.setModel(new JToggleButton.ToggleButtonModel());
itemFour.setSelectedIcon(selectedIcon);
testMenu.add(itemFour);
tBar.add(createToolBarButton(itemFour));
workaround(itemFour, selectedIcon, unselectedIcon);

mBar.add(testMenu);
sMainFrame.setJMenuBar(mBar);

JLabel lbl = new JLabel(System.getProperty("java.version"));
add(lbl, BorderLayout.SOUTH);
}

private void workaround(final JMenuItem menuItem, final Icon selectedIcon, final Icon unselectedIcon) {
ChangeListener cl = new ChangeListener() {
public void stateChanged(ChangeEvent e) {
menuItem.setIcon(menuItem.getModel().isSelected()? selectedIcon : unselectedIcon);
}
};
menuItem.getModel().addChangeListener(cl);
}

private AbstractButton createToolBarButton(JMenuItem item) {
JButton btn = new JButton(item.getText(), item.getIcon());
btn.setSelectedIcon(item.getSelectedIcon());
btn.setModel(item.getModel());
return btn;
}

private class UnselectedIcon implements Icon {
protected Color getLineColor() { return Color.gray; }
public void paintIcon(Component c, Graphics g, int x, int y) {
g.setColor(getLineColor());
int x1 = x+2;
int x2 = x + getIconWidth() - 2;
int y1 = y+2;
int y2 = y + getIconHeight() -2;
g.drawLine(x1, y1, x2, y2);
g.drawLine(x1, y2, x2, y1);
}

public int getIconWidth() { return 16; }
public int getIconHeight() { return getIconWidth(); }
}

private class SelectedIcon extends UnselectedIcon {
@Override protected Color getLineColor() { return Color.red; }
}
}

---------- END SOURCE ----------

CUSTOMER SUBMITTED WORKAROUND :
Write a change listener that sets the unselected icon to the desired
selectedIcon when the state changes to selected. This is done in
menu items 3 and 4. Note that in JDK 1.5, the button text also draws
in red when the icons are red. This is fixed in JDK 1.6.