xul:slider accessible of xul:scale is accessible illegally

VERIFIED FIXED

Status

()

defect
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

(Blocks 1 bug, {access, verified1.9.1})

unspecified
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Assignee

Description

10 years ago
it's happen with xul:scale accessible which contains xul:slider element, we should check this case. This prevents AT to grab name from slider accessible of video element (see bug 490287).
Assignee

Comment 1

10 years ago
changing summary to reflect problem better
Summary: focus event can be fired for non accessible anonymous content → xul:slider accessible of xul:scale is accessible illegally
Assignee

Comment 2

10 years ago
Ok, we create because xul:scale allows accessibles in anonymous content because of xul:thumb (btw, see bug 492513) and generic accessible for xul:slider is created because it is focusable.
Assignee

Comment 3

10 years ago
Posted patch patchSplinter Review
Attachment #377104 - Flags: review?(marco.zehe)
Attachment #377104 - Flags: review?(david.bolter)
Assignee

Comment 4

10 years ago
disable accessible in anonymous content, including thumb accessibles - it sounds slider accessible lives successfully without thumb accessibles (bug 492513).
Attachment #377104 - Flags: review?(david.bolter) → review+
Comment on attachment 377104 [details] [diff] [review]
patch

r=me (unless the vendors disagree re: email) for the non-test part. I'm still not quite sure how the test covers the bug.
OK IRC chat made me happy for the test part :)
Assignee

Comment 8

10 years ago
checked in http://hg.mozilla.org/mozilla-central/rev/768064b2ee15 without AT approval because any way it's pure crossplatform fix which doesn't affect on AT and it makes HTML 5 audio/video more accessible.

Let's move slider accessibility discussions to bug 492513.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Assignee

Updated

10 years ago
Flags: in-testsuite+
Assignee

Updated

10 years ago
Attachment #377104 - Flags: approval1.9.1?
Assignee

Comment 9

10 years ago
Comment on attachment 377104 [details] [diff] [review]
patch

needed for HTML5 audio/video accessibility
Comment on attachment 377104 [details] [diff] [review]
patch

a191=beltzner
Attachment #377104 - Flags: approval1.9.1? → approval1.9.1+
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090517 Minefield/3.6a1pre (.NET CLR 3.5.30729)
Status: RESOLVED → VERIFIED
Verified fixed on 1.9.1 in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b5pre) Gecko/20090517 Shiretoko/3.5b5pre (.NET CLR 3.5.30729)
You need to log in before you can comment on or make changes to this bug.