Name: gm110360 Date: 02/03/2003


FULL PRODUCT VERSION :
1.4.1_01-b01

FULL OPERATING SYSTEM VERSION :
Microsoft Windows XP [Version 5.1.2600]

EXTRA RELEVANT SYSTEM CONFIGURATION :
Metal look and feel

A DESCRIPTION OF THE PROBLEM :
Under the Metal look and feel, if a JMenuItem has an icon
whose paintIcon method calls Graphics.setColor, the last
color set is used to render the menu item text, rather
than the correct text color. The program below
demonstrates this problem.

STEPS TO FOLLOW TO REPRODUCE THE PROBLEM :
1. Run the program below.
2.
3.

REPRODUCIBILITY :
This bug can be reproduced always.

---------- BEGIN SOURCE ----------
import java.awt.Color;
import java.awt.Component;
import java.awt.Graphics;
import javax.swing.*;

/**
 * This program demonstrates a JMenuItem bug under the Metal look and feel
 * under Java 1.4.1 for Windows. A JMenuItem can render its text with the
 * wrong color if it has an Icon whose paintIcon method calls
 * Graphics.setColor.
 *
 * Harold Mills
 * Cornell Lab of Ornithology
 * 3 February 2003
 */
public class JMenuItemBug {

    public static void main(String[] inArgs) throws Exception {

        // menu item
        JMenuItem item = new JMenuItem("This text should not be red.");
        item.setIcon(new ColorIcon(Color.red, 24, 12));

        // menu
        JMenu menu = new JMenu("Select Me");
        menu.add(item);

        // menu bar
        JMenuBar menuBar = new JMenuBar();
        menuBar.add(menu);

        // frame
        JFrame frame = new JFrame("JMenuItem Bug Demo");
        frame.setSize(300, 200);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setJMenuBar(menuBar);
        frame.setVisible(true);

    }

    private static class ColorIcon implements Icon {

        private Color mColor;
        private int mWidth;
        private int mHeight;

        public ColorIcon(Color inColor, int inWidth, int inHeight) {
            mColor = inColor;
            mWidth = inWidth;
            mHeight = inHeight;
        }

        public int getIconWidth() {
            return mWidth;
        }

        public int getIconHeight() {
            return mHeight;
        }

        public void paintIcon(
            Component inComponent,
            Graphics inGraphics,
            int inX,
            int inY
        ) {

            // Here we call Graphics.setColor as part of our icon rendering.
            // This color will later be used to render the menu item text,
            // a bug.
            inGraphics.setColor(mColor);

            inGraphics.fillRect(inX, inY, mWidth, mHeight);

        }

    }

}

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

CUSTOMER WORKAROUND :
Call Graphics.setColor at the end of Icon.paintIcon with
the correct text color.

Release Regression From : 1.3.1_07
The above release value was the last known release where this
bug was known to work. Since then there has been a regression.

(Review ID: 180721)
======================================================================
###@###.### 10/12/04 14:16 GMT
[SQE]
RISK: LOW
Regtest: test/javax/swing/JMenuItem/4812431/bug4812431.java
Testing on any one platform is suffixient, fix affects painting of the text in menu items in one specific LAF (Metal).
[SQE]