overflowed content doesn't expose child text accessibles

VERIFIED FIXED

Status

()

defect
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: surkov, Assigned: surkov)

Tracking

(Blocks 1 bug, {fixed1.9.1})

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

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

Assignee

Description

10 years ago
<div style="overflow: auto">long content</div> doesn't expose child text accessible. I suspect bug 483573 because there we introduced walking through frames if there is anonymous content. I have wip patch that fixes the problem (fix logic of a11y tree walker) and will post it once I'll polish it.
Assignee

Comment 1

10 years ago
Posted patch patchSplinter Review
short description: when I added querying to anonymous content creator then a11y tree walker founds scrollbar element in the case of overflowed div, scrollbar isn't accessible so it checks it first child , GetFirstChild is called again and then GetKids. GetKids sees it's XUL element so it nulls frame of current state before we put the current state on stack.
Attachment #373824 - Flags: review?(david.bolter)
Assignee

Updated

10 years ago
Attachment #373824 - Flags: review?(marco.zehe)
Assignee

Updated

10 years ago
Status: NEW → ASSIGNED
Comment on attachment 373824 [details] [diff] [review]
patch

>+   * DOM/frame subtree duiring searching of accessible children.

Nit: Typo "diuring" instead of "during".

>+   * Pop state from stack and makes it current.

Nit: Either "pops state...and makes..." or "pop state...and make it..." I'd prefer the latter (the one with pop and make).
Attachment #373824 - Flags: review?(marco.zehe) → review+
Comment on attachment 373824 [details] [diff] [review]
patch

r=me
Attachment #373824 - Flags: review?(david.bolter) → review+
Assignee

Comment 4

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

must be landed with bug 483573
Attachment #373824 - Flags: approval1.9.1?
Assignee

Comment 5

10 years ago
checked in http://hg.mozilla.org/mozilla-central/rev/81273d925ba9
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Comment on attachment 373824 [details] [diff] [review]
patch

a191=beltzner
Attachment #373824 - Flags: approval1.9.1? → approval1.9.1+
Assignee

Comment 7

10 years ago
Posted patch patch 1.9.1Splinter Review
Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090503 Minefield/3.6a1pre (.NET CLR 3.5.30729)
Status: RESOLVED → VERIFIED
Assignee

Updated

10 years ago
Blocks: 492516
You need to log in before you can comment on or make changes to this bug.