526915 - aria-describedby broken in FF 3.6b1

Status: VERIFIED FIXED

(Core :: Disability Access APIs, defect)

Description

[Hans Hillen]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2b1) Gecko/20091029 Firefox/3.6b1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2b1) Gecko/20091029 Firefox/3.6b1

Descriptive text added to an element using aria-describedby is not exposed as MSAA description.

Reproducible: Always

Steps to Reproduce:
Use attached sample. It contains a non-functional ARIA button and text that describes the button. 
Tab to the button so that it receives focus.
Actual Results:  
Descriptive text is not announced by NVDA and  the 'description' field is empty in MSAA inspect tools. Interestingly enough, JAWS 11 still reads the description despite it not being exposed in MSAA

Expected Results:  
The descriptive text target with aria-describedby is exposed as MSAA description, as it did in FF3.5

Comment 1

[Hans Hillen]

Attachment: aria-describedby testcase

Comment 2

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

Marco, if possible I'd like a regression window.

Comment 3

[Marco Zehe (:MarcoZ)]

This is confirmed for this specific testcase. However, my original Easy ARIA Tip #2 from http://www.marcozehe.de/2008/03/23/easy-aria-tip-2-aria-labelledby-and-aria-describedby/ still works, so this is something specific to this testcase.

Comment 4

[Marco Zehe (:MarcoZ)]

This was introduced in the following build:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2a1pre) Gecko/20090219 Minefield/3.2a1pre (.NET CLR 3.5.30729)
The build from February 18 is still OK.

Comment 5

[alexander :surkov (:asurkov)]

Attachment: patch

Comment 6

[Marco Zehe (:MarcoZ)]

Comment on attachment 410734 [details] [diff] [review]
patch

r=me. Thanks!

Comment 7

[alexander :surkov (:asurkov)]

(In reply to comment #6)
> (From update of attachment 410734 [details] [diff] [review])
> r=me. Thanks!

I'm really going to love our name rules map, it's getting pretty easy to fix bugs :)

Comment 8

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

Comment on attachment 410734 [details] [diff] [review]
patch

r=me, nice to see the name rules pay off :)

Comment 9

[alexander :surkov (:asurkov)]

landed on 1.9.3 - http://hg.mozilla.org/mozilla-central/rev/b697b641e508

Comment 10

[Marco Zehe (:MarcoZ)]

This is a regression from Firefox 3.5 and will impact all web authors trying to USE aria-describedby with certain constructs.

Comment 11

[Marco Zehe (:MarcoZ)]

Comment on attachment 410734 [details] [diff] [review]
patch

Safe fix, has tests.

Comment 12

[Marco Zehe (:MarcoZ)]

Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.3a1pre) Gecko/20091110 Minefield/3.7a1pre (.NET CLR 3.5.30729)

Comment 13

[Marco Zehe (:MarcoZ)]

Pushed to 1.9.2 on Alexander's behalf:
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/b0c3db90958d

Comment 14

[alexander :surkov (:asurkov)]

(In reply to comment #13)
> Pushed to 1.9.2 on Alexander's behalf:
> http://hg.mozilla.org/releases/mozilla-1.9.2/rev/b0c3db90958d

Thank you!

Comment 15

[Marco Zehe (:MarcoZ)]

Verified fixed in Mozilla/5.0 (Windows; U; Windows NT 6.1; de; rv:1.9.2b3) Gecko/20091115 Firefox/3.6b3 (.NET CLR 3.5.30729)