288517 - Accessible progress meter fixes

Status: RESOLVED FIXED

(Firefox :: Disability Access, defect)

Description

[Aaron Leventhal]

Progress meters need to fire accessible value change events and they are not
focusable.

Comment 1

[Aaron Leventhal]

Attachment: Simple patch to fix state and fire valuechange events from xbl

The toolkit stuff doesn't need to be sr='d, but the rest does.

Comment 2

[[e-mail]]

Comment on attachment 179209 [details] [diff] [review]
Simple patch to fix state and fire valuechange events from xbl

Bah, this was one of the code fragments that I thought we could get rid of now
that the attribute setting code is smarter :-/

Comment 3

[Aaron Leventhal]

> now that the attribute setting code is smarter :-/
What's smarter about it? 

Comment 4

[[e-mail]]

Don't mind me, I thought we were dealing with integers.

Comment 5

[Aaron Leventhal]

Checking in accessible/src/xul/nsXULFormControlAccessible.cpp;
/cvsroot/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp,v  <-- 
nsXULFormControlAccessible.cpp
new revision: 1.47; previous revision: 1.46
done
Checking in xpfe/global/resources/content/bindings/progressmeter.xml;
/cvsroot/mozilla/xpfe/global/resources/content/bindings/progressmeter.xml,v  <--
 progressmeter.xml
new revision: 1.13; previous revision: 1.12
done
Checking in toolkit/content/widgets/progressmeter.xml;
/cvsroot/mozilla/toolkit/content/widgets/progressmeter.xml,v  <--  progressmeter.xml
new revision: 1.6; previous revision: 1.5
done

Comment 6

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1527381