Issue 843594 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	----
Closed:	May 2018
Cc:	dmazz...@chromium.org
EstimatedDays:	----
NextAction:	----
OS:	Mac
Pri:	3
Type:	Bug

Don't ignore nodes with any aria-foo attrbiute

Project Member Reported by aleventhal@chromium.org, May 16 2018

Issue description

Rather than whitelisting attributes to check in AXLayoutObject::ComputeAccessibilityIsIgnored (like aria-details), we could iterate over all the attributes and not ignore an element if it has any aria-foo attribute.

Project Member

Comment 1 by bugdroid1@chromium.org, May 16 2018

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

commit 3f72dc8ded70719c578e3643e58d67c631255756
Author: Aaron Leventhal <aleventhal@chromium.org>
Date: Wed May 16 22:28:06 2018

Any ARIA attribute should cause a node to be exposed

Bug:  843594 
Change-Id: I47910f9634b2a66e8ca979427ad1d42cb45ffd46
Reviewed-on: https://chromium-review.googlesource.com/1062407
Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#559313}
[modify] https://crrev.com/3f72dc8ded70719c578e3643e58d67c631255756/content/browser/accessibility/dump_accessibility_tree_browsertest.cc
[add] https://crrev.com/3f72dc8ded70719c578e3643e58d67c631255756/content/test/data/accessibility/aria/aria-any-unignored-expected-blink.txt
[add] https://crrev.com/3f72dc8ded70719c578e3643e58d67c631255756/content/test/data/accessibility/aria/aria-any-unignored.html
[modify] https://crrev.com/3f72dc8ded70719c578e3643e58d67c631255756/third_party/blink/renderer/modules/accessibility/ax_layout_object.cc

Comment 2 by aleventhal@chromium.org, May 16 2018

Status: Fixed (was: Available)

►

Issue 843594 link

Issue metadata

Don't ignore nodes with any aria-foo attrbiute

Issue description

Comment 1 by bugdroid1@chromium.org, May 16 2018

Comment 1 Deleted

Comment 2 by aleventhal@chromium.org, May 16 2018

Comment 2 Deleted

Sign in to add a comment