[cp-patches] FYI: AccessibleAbstractButton fixlet

[Roman Kennke]

I should have looked more closely at the AccessibleAWTComponent when
implementing AccessibleAbstractButton.getAccessibleStateSet(). For some
reason I duplicated the handling of the focused state there, which is
not necessary. I  removed this.

2006-04-05  Roman Kennke  <[EMAIL PROTECTED]>

        * javax/swing/AbstractButton.java
        (AccessibleAbstractButton.getAccessibleStateSet): Removed handling
        of the focused state. This is already done in
AccessibleAWTComponent.

/Roman

Index: javax/swing/AbstractButton.java
===================================================================
RCS file: /cvsroot/classpath/classpath/javax/swing/AbstractButton.java,v
retrieving revision 1.54
diff -u -1 -0 -r1.54 AbstractButton.java
--- javax/swing/AbstractButton.java	5 Apr 2006 16:23:56 -0000	1.54
+++ javax/swing/AbstractButton.java	5 Apr 2006 21:50:46 -0000
@@ -440,22 +440,20 @@
     public AccessibleStateSet getAccessibleStateSet()
     {
       AccessibleStateSet state = super.getAccessibleStateSet();
 
       if (getModel().isArmed())
         state.add(AccessibleState.ARMED);
       if (getModel().isPressed())
         state.add(AccessibleState.PRESSED);
       if (isSelected())
         state.add(AccessibleState.CHECKED);
-      if (isFocusOwner())
-        state.add(AccessibleState.FOCUSED);
 
       return state;
     }
 
     /**
      * Returns the accessible name for the button.
      */
     public String getAccessibleName()
     {
       String result = super.getAccessibleName();