Bug 288517. Fix accessibility API support of progress meters. r=mconnor, sr=neil

accessible/src/xul/nsXULFormControlAccessible.cpp

@@ -451,9 +451,11 @@ NS_IMETHODIMP nsXULProgressMeterAccessible::GetRole(PRUint32 *_retval)
 /**
   * No states supported for progressmeter
   */
-NS_IMETHODIMP nsXULProgressMeterAccessible::GetState(PRUint32 *_retval)
+NS_IMETHODIMP nsXULProgressMeterAccessible::GetState(PRUint32 *aState)
 {
-  return nsAccessible::GetState(_retval);
+  nsresult rv = nsAccessible::GetState(aState);
+  *aState &= ~STATE_FOCUSABLE;
+  return rv;
 }
 
 NS_IMETHODIMP nsXULProgressMeterAccessible::GetValue(nsAString& _retval)

toolkit/content/widgets/progressmeter.xml

@@ -30,8 +30,13 @@
             var delta = p - c;
             if (delta < 0)
               delta = -delta;
-            if (delta > 3 || p == 0 || p == 100)
+            if (delta > 3 || p == 0 || p == 100) {
               this.setAttribute("value", p);
+              // Fire DOM event so that accessible value change events occur
+              var event = document.createEvent('Events');
+              event.initEvent('ValueChange', true, true);
+              this.dispatchEvent(event);
+            }
           }
           
           return p;

xpfe/global/resources/content/bindings/progressmeter.xml

@@ -30,8 +30,13 @@
             var delta = p - c;
             if (delta < 0)
               delta = -delta;
-            if (delta > 3 || p == 0 || p == 100)
+            if (delta > 3 || p == 0 || p == 100) {
               this.setAttribute("value", p);
+              // Fire DOM event so that accessible value change events occur
+              var event = document.createEvent('Events');
+              event.initEvent('ValueChange', true, true);
+              this.dispatchEvent(event);
+            }
           }
           
           return p;