1198336 - Firefox/Voiceover: aria-live regions not being announced

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect, P1)

Description

[Danny Wieser]

Attachment: aria-live.html

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/44.0.2403.157 Safari/537.36

Steps to reproduce:

Within a dynamic web application, attempt to utilize an aria-live region to announce either status information.

* Aria-live region populated by using javascript to append child elements to the live region
* Assertiveness set to polite (although issue also applies to other settings)

Mac 10.10.5, Voiceover and Firefox 40.0


Actual results:

Voiceover reads nothing from the aria-live region.


Expected results:

When a new child node is appended to the aria live region, the screenreader should announce the text value of that node. 

Note that Windows 7/Firefox 40.0.2/JAWS 16 works as expected.

Comment 1

[Marco Zehe (:MarcoZ)]

Known issue that has to do with our as of yet incomplete implementation of Mac accessibility APIs.

Comment 2

[Florian Bender]

Is this bug still valid/reproducible?

Comment 3

[Marco Zehe (:MarcoZ)]

Very likely yes. We haven't done any work on Mac a11y in recent months due to resourcing/prioritization, and thus haven't actively pursued this or other Mac-related bugs.

Comment 4

[Morgan Reschenberg [:morgan]]

Verified today, still an issue.

Comment 5

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1198336 - P1: Introduce live region added/removed events. r?Jamie!,morgan!

VoiceOver pre-caches live region data and does its own deltas to
know what to parts of a subtree changed, and what to announce
based on AXARIAAtomic and AXARIARelevant.

I added a removed event as well. This will help us cache a "live region"
flag in the main process and avoid sync round trips for attributes when not needed.

Comment 6

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1198336 - P2: Implement live region AX attributes and AXLiveRegionCreated notification. r?morgan!

We use an ivar boolean flag to cache the existance of a live region. This lets us
hide the live region AX attributes and avoid sync round trips to content.

Depends on D96291

Comment 7

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1198336 - P3: Implement AXLiveRegionSearchKey. r?morgan!

I think VoiceOver uses this to pull in all live regions so it
can check for deletions when they change later on.

Depends on D96292

Comment 8

[Eitan Isaacson [:eeejay]]

Attachment: Bug 1198336 - P4: Post AXLiveRegionChanged when live regions change. r?morgan!

VoiceOver seems to do all the heavy lifting of figuring out what portion of the
live region should be read.

Depends on D96293

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/68cf7e4b16f2
P1: Introduce live region added/removed events. r=Jamie,morgan
https://hg.mozilla.org/integration/autoland/rev/fd45661dd659
P2: Implement live region AX attributes and AXLiveRegionCreated notification. r=morgan
https://hg.mozilla.org/integration/autoland/rev/7a2dbe67b6a6
P3: Implement AXLiveRegionSearchKey. r=morgan
https://hg.mozilla.org/integration/autoland/rev/8399406e5f2e
P4: Post AXLiveRegionChanged when live regions change. r=morgan

Comment 10

[Atila Butkovits]

Backed out for bustages on rules.mk.

Backout link: https://hg.mozilla.org/integration/autoland/rev/f320b3be066b9443a63a340e05eb1c03f0dc47c5

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&revision=8399406e5f2e7a692318745af59b31ebe7df6316&searchStr=build&selectedTaskRun=dRjrRkPCQM-_di10voVhKQ.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=321395989&repo=autoland&lineNumber=13375

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2c56e36b869d
P1: Introduce live region added/removed events. r=Jamie,morgan
https://hg.mozilla.org/integration/autoland/rev/ee78c1e8cc3b
P2: Implement live region AX attributes and AXLiveRegionCreated notification. r=morgan
https://hg.mozilla.org/integration/autoland/rev/821f94bae182
P3: Implement AXLiveRegionSearchKey. r=morgan
https://hg.mozilla.org/integration/autoland/rev/faa35ea77d35
P4: Post AXLiveRegionChanged when live regions change. r=morgan

Comment 12

[Natalia Csoregi [:nataliaCs]]

Backed out for causing leaks

backout: https://hg.mozilla.org/integration/autoland/rev/79670befed2a2a89e2c0b0dc73403212fbb43738

push: https://treeherder.mozilla.org/jobs?repo=autoland&revision=faa35ea77d35aeea0be2fe968cc38a8a13a57407&group_state=expanded&selectedTaskRun=DsP73xzZSHSemPHeOdx0Wg.0

failure log: https://treeherder.mozilla.org/logviewer?job_id=321502589&repo=autoland&lineNumber=5237

Comment 13

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/67f5a42ed170
P1: Introduce live region added/removed events. r=Jamie,morgan
https://hg.mozilla.org/integration/autoland/rev/823511f4191b
P2: Implement live region AX attributes and AXLiveRegionCreated notification. r=morgan
https://hg.mozilla.org/integration/autoland/rev/852dc98e8cf5
P3: Implement AXLiveRegionSearchKey. r=morgan
https://hg.mozilla.org/integration/autoland/rev/26ad5bb8e1ed
P4: Post AXLiveRegionChanged when live regions change. r=morgan

Comment 14

[Atila Butkovits]

Backed out for bustage on rules.mk.

Backout link: https://hg.mozilla.org/integration/autoland/rev/4608f0501f6c9a033e8e2531ab364a9e4399cd2d

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=26ad5bb8e1ed1d4a9dfe3a7007158bc09cf936f0&searchStr=os%2Cx%2Ccross%2Ccompiled%2Cshippable%2Copt%2Cprofile-guided%2Coptimization%2Cbuilds%2Cinstrumented-build-macosx64-shippable%2Fopt%2Cinstr&selectedTaskRun=Jc2sIyImRoO0e1L8GcvKbw.0

Failure log: https://treeherder.mozilla.org/logviewer?job_id=321674892&repo=autoland&lineNumber=39596

Comment 15

[Eitan Isaacson [:eeejay]]

Ah sorry. Added an include, and tested on try. I think we're good:
https://treeherder.mozilla.org/jobs?repo=try&revision=60912d72cd9119886d5a431c5a9e446918aeafce

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d5ba14136bc2
P1: Introduce live region added/removed events. r=Jamie,morgan
https://hg.mozilla.org/integration/autoland/rev/f1d53dd3f6a5
P2: Implement live region AX attributes and AXLiveRegionCreated notification. r=morgan
https://hg.mozilla.org/integration/autoland/rev/984c4b41ba37
P3: Implement AXLiveRegionSearchKey. r=morgan
https://hg.mozilla.org/integration/autoland/rev/304951c0df44
P4: Post AXLiveRegionChanged when live regions change. r=morgan

Comment 17

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/d5ba14136bc2
https://hg.mozilla.org/mozilla-central/rev/f1d53dd3f6a5
https://hg.mozilla.org/mozilla-central/rev/984c4b41ba37
https://hg.mozilla.org/mozilla-central/rev/304951c0df44