Closed Bug 492518 Opened 16 years ago Closed 16 years ago

xul:slider accessible of xul:scale is accessible illegally

Categories

(Core :: Disability Access APIs, defect)

Product:
Core
Component:
Disability Access APIs
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: surkov, Assigned: surkov)

References

(Blocks 1 open bug)

Details

(Keywords: access, verified1.9.1)

Attachments

(1 file)

patch
(deleted), patch
MarcoZ
: review+
davidb
: review+
beltzner
: approval1.9.1+
Details | Diff | Splinter Review
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).
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
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.
Attached patch patch (deleted) — — Splinter Review
Attachment #377104 - Flags: review?(marco.zehe)
Attachment #377104 - Flags: review?(david.bolter)
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 :)
Comment on attachment 377104 [details] [diff] [review] patch r=me as well.
Attachment #377104 - Flags: review?(marco.zehe) → review+
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
Closed: 16 years ago
Resolution: --- → FIXED
Flags: in-testsuite+
Attachment #377104 - Flags: approval1.9.1?
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)
Keywords: fixed1.9.1 → verified1.9.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: