873447 - expose IAccessible2 checkable state

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

IA2_STATE_CHECKABLE (a plain mapping)

Comment 1

[alexander :surkov (:asurkov)]

(In reply to alexander :surkov from comment #0)
> IA2_STATE_CHECKABLE (a plain mapping)

map states::CHECKED into IA2_STATE_CHECKABLE (msaa/AccessibleWrap::get_states)

Comment 2

[MikeLing]

Hi I am interested in working on this bug,but it's my first time to work on with debug,can anybody guide me on how to get started with it?Thanks a lot.

Comment 3

[David Bolter [:davidb] (NeedInfo me for attention)]

(In reply to MikeLing from comment #2)
> Hi I am interested in working on this bug,but it's my first time to work on
> with debug

Thanks! What do you mean by 'with debug'?

Comment 4

[Marcos A. Di Pietro]

I have a patch for this, please go ahead and assign me this bug.

Comment 5

[alexander :surkov (:asurkov)]

(In reply to Marcos A. Di Pietro from comment #4)
> I have a patch for this, please go ahead and assign me this bug.

done. I hope it's ok with MikeLing who was interested working on it but we work with him in another bug already, it should be ok

Comment 6

[Marcos A. Di Pietro]

Attachment: Patch v1

Maps states::CHECKED to IA2_STATE_CHECKABLE in msaa/AccessibleWrap.cpp::get_states

Comment 7

[alexander :surkov (:asurkov)]

Comment on attachment 757761 [details] [diff] [review]
Patch v1

Review of attachment 757761 [details] [diff] [review]:
-----------------------------------------------------------------

plain and simple, thank you for the fix!

Comment 8

[Marco Zehe (:MarcoZ)]

Landed: http://hg.mozilla.org/integration/mozilla-inbound/rev/70e5bad55544

Comment 9

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/70e5bad55544