252955 - List items returning strange states

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[Aaron Leventhal]

Steps:
Run MSAA SDK's accessible event watch
View-> Char Encodings -> Customize
Arrow through list

What happens:
Extra states reported

Expected:
A set of sane states

Comment 1

[Aaron Leventhal]

Attachment: Correct unititialized variable situation

Comment 2

[Aaron Leventhal]

Comment on attachment 154256 [details] [diff] [review]
Correct unititialized variable situation

Still needs work. The wrong listitem is getting the focused state and focus
event.

Comment 3

[Aaron Leventhal]

Attachment: Fire correct focus events, make sure focused state is set on list item event targets for them

Comment 4

[Aaron Leventhal]

Comment on attachment 154258 [details] [diff] [review]
Fire correct focus events, make sure focused state is set on list item event targets for them

Darin, this doesn't need to be sr='d for accessibility module correctness, just
for C++ correctness/integration super review.

Comment 5

[Darin Fisher]

Comment on attachment 154258 [details] [diff] [review]
Fire correct focus events, make sure focused state is set on list item event targets for them

Technically, you should really find a non-IBMer to provide SR because it is
"verboten" to have one company both write patches and approve patches for
Mozilla.

We sort-of had the same policy at Netscape... no one-group in Netscape was
permitted to both author and approve patches.

Patch looks clean to me, sr=darin.

Comment 6

[Aaron Leventhal]

Checking in base/nsRootAccessible.cpp;
/cvsroot/mozilla/accessible/src/base/nsRootAccessible.cpp,v  <-- 
nsRootAccessible.cpp
new revision: 1.99; previous revision: 1.98
done
Checking in xul/nsXULSelectAccessible.cpp;
/cvsroot/mozilla/accessible/src/xul/nsXULSelectAccessible.cpp,v  <-- 
nsXULSelectAccessible.cpp
new revision: 1.21; previous revision: 1.20
done

Comment 7

[Firefox Bug Husbandry Bot]

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