expose IAccessible2 checkable state

RESOLVED FIXED in mozilla24

Status

()

Core
Disability Access APIs
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: surkov, Assigned: Marcos A. Di Pietro)

Tracking

(Blocks: 1 bug, {access})

unspecified
mozilla24
access
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mentor=surkov.alexander@gmail.com][lang=c++])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
IA2_STATE_CHECKABLE (a plain mapping)
(Reporter)

Updated

4 years ago
No longer blocks: 614570
Depends on: 614570
(Reporter)

Updated

4 years ago
Blocks: 368873
(Reporter)

Comment 1

4 years ago
(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

4 years ago
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.
(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'?
(Assignee)

Comment 4

4 years ago
I have a patch for this, please go ahead and assign me this bug.
(Reporter)

Comment 5

4 years ago
(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
Assignee: nobody → marcosadp
(Assignee)

Comment 6

4 years ago
Created attachment 757761 [details] [diff] [review]
Patch v1

Maps states::CHECKED to IA2_STATE_CHECKABLE in msaa/AccessibleWrap.cpp::get_states
Attachment #757761 - Flags: review?(surkov.alexander)
(Reporter)

Comment 7

4 years ago
Comment on attachment 757761 [details] [diff] [review]
Patch v1

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

plain and simple, thank you for the fix!
Attachment #757761 - Flags: review?(surkov.alexander) → review+
(Reporter)

Updated

4 years ago
Keywords: checkin-needed

Comment 8

4 years ago
Landed: http://hg.mozilla.org/integration/mozilla-inbound/rev/70e5bad55544
Keywords: checkin-needed

Comment 9

4 years ago
https://hg.mozilla.org/mozilla-central/rev/70e5bad55544
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.