New issue
Advanced search Search tips

Issue 866341 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Sep 11
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

[Orca] Accessible state-changed:checked events missing

Project Member Reported by joanmari...@gmail.com, Jul 23

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3500.0 Safari/537.36

Steps to reproduce the problem:
1. Launch Chrome/Chromium
  * export ACCESSIBILITY_ENABLED=1, and
  * launch with --force-renderer-accessibility

2. Launch the accessible-event listener in a terminal

3. Load "data:text/html,<input type="checkbox" checked/>" in browser

4. Give the input focus

5. Toggle its state with space bar

What is the expected behavior?
The accessible-event listener would print out events for focus changes as well as state changes. In addition, the checkbox would have state "checkable".

What went wrong?
Actual results: The accessible-event listener only prints out events for focus changes. In addition, the checkbox lacks state "checkable."

Did this work before? No 

Chrome version: 70.0.3500.0  Channel: n/a
OS Version: 
Flash Version: 

Compare to: Firefox (with accessibility enabled), Epiphany

Documentation: https://developer.gnome.org/atk/stable/atk-AtkState.html#AtkStateType, https://developer.gnome.org/atk/stable/AtkObject.html#AtkObject.signals

Impact: Because there is no accessible event when the checked state changes, Orca cannot present the change to the user. A user has to give something else focus and then return to the checkbox to verify that its state has changed.
 
checked-events.py
631 bytes View Download
Labels: Needs-Triage-M70
Components: -UI UI>Accessibility
Cc: dmazz...@chromium.org
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 11

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f

commit ce4648fe0c0dddcae87f257ade6256a4cf3ee02f
Author: Joanmarie Diggs <jdiggs@igalia.com>
Date: Tue Sep 11 16:50:14 2018

Emit ATK state-changed events for toggleable widgets on Aura Linux

* Emit checked state changed events for checkboxes and radio buttons.
* Emit pressed state changed events for toggle buttons.
* Emit indeterminate state changed events for tri-state widgets when
  their state becomes "mixed."
* Remove checkable state for toggle buttons.

R=dmazzoni@google.com

Bug:  866341 
Change-Id: I10e08330408c6f783e202d2c81c6c70d19309d08
Reviewed-on: https://chromium-review.googlesource.com/1219726
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Joanmarie Diggs <joanmarie.diggs@gmail.com>
Cr-Commit-Position: refs/heads/master@{#590350}
[modify] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/browser/accessibility/browser_accessibility_manager_auralinux.cc
[modify] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/browser/accessibility/browser_accessibility_manager_auralinux.h
[modify] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/browser/accessibility/dump_accessibility_events_browsertest.cc
[add] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/test/data/accessibility/event/aria-checked-changed-expected-auralinux.txt
[add] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/test/data/accessibility/event/aria-checked-changed.html
[add] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/test/data/accessibility/event/aria-pressed-changed-expected-auralinux.txt
[add] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/test/data/accessibility/event/aria-pressed-changed.html
[add] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/content/test/data/accessibility/event/checked-state-changed-expected-auralinux.txt
[modify] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/ui/accessibility/platform/ax_platform_node_auralinux.cc
[modify] https://crrev.com/ce4648fe0c0dddcae87f257ade6256a4cf3ee02f/ui/accessibility/platform/ax_platform_node_auralinux.h

Status: Fixed (was: Unconfirmed)
Closing as fixed.

Sign in to add a comment