Table cell renderer works with any L&F, but not with Windows

Question

Hi,
I created a table cell renderer / editor for a combobox in a table cell. This works all as expected, but if the Look&Feel is com.sun.java.swing.plaf.windows.WindowsLookAndFeel, which is the system L&F on Vista, the selected values in the combobox are not shown in the cell after loosing focus. Metal LF and Motif LF work, and a third party one "TinyLF" works as well.
Can anyone imagine why, and what needs to be changed to get a cross platform consistency without Metal LF?
A compilable demo:
import java.awt.BorderLayout;
import java.awt.Component;
import java.io.IOException;
import java.security.NoSuchAlgorithmException;
import javax.swing.DefaultCellEditor;
import javax.swing.DefaultComboBoxModel;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JTable;
import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;
import javax.swing.table.DefaultTableModel;
import javax.swing.table.TableCellRenderer;
import javax.swing.table.TableColumn;
import javax.swing.table.TableModel;
public class Test {
    static class CellRendererWithComboBox extends JLabel implements TableCellRenderer {
        JLabel label = new JLabel();
         * Set this renderer to the given column
         * @param column
         * @param r
         * @param editable
        public void setRendererTo(JTable table, int column, boolean editable) {
            TableColumn col = table.getColumnModel().getColumn(column);
            JComboBox xc = new JComboBox(new DefaultComboBoxModel(new Object[]{1, 2, 3, 4, 5, 6, 7}));
            col.setCellEditor(new ComboBoxEditor(xc));
            col.setCellRenderer(this);
            xc.setEditable(editable);
        @Override
        public Component getTableCellRendererComponent(JTable table, Object value,
                boolean isSelected, boolean hasFocus, int row, int column) {
            if (hasFocus && isSelected) {
                if (isSelected) {
                    label.setForeground(table.getSelectionForeground());
                    label.setBackground(table.getSelectionBackground());
                } else {
                    label.setForeground(table.getForeground());
                    label.setBackground(table.getBackground());
                label.setText((value == null) ? "" : value.toString());
                return label;
            } else {
                label.setText((value == null) ? "" : value.toString());
                return label;
        class ComboBoxEditor extends DefaultCellEditor {
            private final JComboBox box;
            private boolean fire = true;
            public ComboBoxEditor(JComboBox b) {
                super(b);
                this.box = b;
                b.setLightWeightPopupEnabled(false);
            @Override
            public boolean stopCellEditing() {
                if (fire) {
                    super.stopCellEditing();
                return true;
            public void stopCellEditingSilent() {
                fire = false;
                stopCellEditing();
                fire = true;
    public static void main(String... aArgs) throws NoSuchAlgorithmException, IOException {
        try {
            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        } catch (ClassNotFoundException classNotFoundException) {
        } catch (InstantiationException instantiationException) {
        } catch (IllegalAccessException illegalAccessException) {
        } catch (UnsupportedLookAndFeelException unsupportedLookAndFeelException) {
        JFrame p = new JFrame();
        JTable t = new JTable();
        TableModel m = new DefaultTableModel(new Object[][]{
                    new Object[]{1, 2, 3, 4, 5, 6, 7},
                    new Object[]{1, 2, 3, 4, 5, 6, 7},
                    new Object[]{1, 2, 3, 4, 5, 6, 7},
                    new Object[]{1, 2, 3, 4, 5, 6, 7},
                    new Object[]{1, 2, 3, 4, 5, 6, 7}},
                new Object[]{1, 2, 3, 4, 5, 6, 7});
        t.setModel(m);
        Test.CellRendererWithComboBox f = new CellRendererWithComboBox();
        f.setRendererTo(t, 1, true);
        p.setLayout(new BorderLayout());
        p.add(t, BorderLayout.CENTER);
        p.pack();
        p.setVisible(true);
}

Answer 1

1) Your custom cell renderer does nothing extra, so you can depend on default renderer.
2) Instead of extending JLabel and implementing TableCellReneder, you should extend DefaultTableCellRenderer and override its get renderer method because DefaultTableCellRenderer itself extends JLabel and aditionally it does some optimizations etc., which you can miss if you implement your own renderer.
3) If you set foreground and background color in last else statement also, then you can see the values correctly in windows L&F as well.
Thanks!