New issue
Advanced search Search tips

Issue 746621 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 3
Type: Bug

Blocked on:
issue 746587



Sign in to add a comment

Ensure invalid ARIA values are mapped appropriately

Project Member Reported by aleventhal@chromium.org, Jul 19 2017

Issue description

CORE AAM says:
If a WAI-ARIA property contains an unknown or disallowed value, the user agent should expose to platform accessibility APIs as follows:
1. When exposing as an object attribute, expose the unknown value — do not vet it against possible values.
2. When exposing as a platform API Boolean state:
  A. For values of "" (empty string), "undefined" or no attribute present:
     - Follow the guidance in the Default values for missing required attributes table below, if applicable.
     - Otherwise, treat as false.
  B. Treat any other value as true.
2. Otherwise, ignore the value and treat the property as not present.
 
Blockedon: 746587
Description: Show this description
Description: Show this description
Project Member

Comment 4 by bugdroid1@chromium.org, Aug 2 2017

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

commit e2ef7518962c8a6b13f07545430f4ff54f9f900e
Author: Aaron Leventhal <aleventhal@chromium.org>
Date: Wed Aug 02 05:06:18 2017

Follow CORE-AAM rules for exposing illegal ARIA values

ARIA booleans that are not "false" and not explicitly undefined should be exposed as true.
ARIA object attributes that expose token should expose the error values and let the AT decide what to do with them. (This enables experimentation with new ARIA experiences, without changing the shipping browser code).

Bug:  746621 
Change-Id: I91381f17db5ad179f26b02b81434b36f78492f0d
Reviewed-on: https://chromium-review.googlesource.com/584018
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#491273}
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/browser/accessibility/dump_accessibility_tree_browsertest.cc
[add] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-illegal-val-expected-blink.txt
[add] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-illegal-val-expected-win.txt
[add] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-illegal-val.html
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-switch-expected-android.txt
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-switch-expected-blink.txt
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-switch-expected-mac.txt
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/content/test/data/accessibility/aria/aria-switch-expected-win.txt
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/third_party/WebKit/Source/core/dom/AccessibleNode.cpp
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/third_party/WebKit/Source/modules/accessibility/AXNodeObject.cpp
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/third_party/WebKit/Source/modules/accessibility/AXObject.cpp
[modify] https://crrev.com/e2ef7518962c8a6b13f07545430f4ff54f9f900e/third_party/WebKit/Source/modules/accessibility/AXTableCell.cpp

Status: Fixed (was: Assigned)

Sign in to add a comment