[Bug swing/25473] New: JFilerChooser doesn't work with an accessory set

[ctl at highsorcery dot com]

For JFileChooser, the Basic UI delegate doesn't initialize the accessoryPanel,
and the Metal UI delegate isn't adding it to the UI. This patch fixes both:

Index: javax/swing/plaf/basic/BasicFileChooserUI.java
===================================================================
RCS file:
/sources/classpath/classpath/javax/swing/plaf/basic/BasicFileChooserUI
.java,v
retrieving revision 1.21
diff -u -r1.21 BasicFileChooserUI.java
--- javax/swing/plaf/basic/BasicFileChooserUI.java      30 Nov 2005 19:42:40
-00
00      1.21
+++ javax/swing/plaf/basic/BasicFileChooserUI.java      18 Dec 2005 00:17:48
-00
00
@@ -669,7 +669,7 @@
   JButton accept;

   /** An optional accessory panel. */
-  JPanel accessoryPanel;
+       JPanel accessoryPanel = new JPanel();

   /** A property change listener. */
   PropertyChangeListener propertyChangeListener;
Index: javax/swing/plaf/metal/MetalFileChooserUI.java
===================================================================
RCS file:
/sources/classpath/classpath/javax/swing/plaf/metal/MetalFileChooserUI
.java,v
retrieving revision 1.13
diff -u -r1.13 MetalFileChooserUI.java
--- javax/swing/plaf/metal/MetalFileChooserUI.java      30 Nov 2005 19:42:41
-00
00      1.13
+++ javax/swing/plaf/metal/MetalFileChooserUI.java      18 Dec 2005 00:17:49
-00
00
@@ -1354,6 +1354,8 @@
     buttonPanel.add(cancelButton);
     bottomPanel.add(buttonPanel);
     fc.add(bottomPanel, BorderLayout.SOUTH);
+
+       fc.add(getAccessoryPanel(), BorderLayout.EAST);
   }

   /**


-- 
           Summary: JFilerChooser doesn't work with an accessory set
           Product: classpath
           Version: 0.20
            Status: UNCONFIRMED
          Severity: critical
          Priority: P3
         Component: swing
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: ctl at highsorcery dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25473