New issue
Advanced search Search tips

Issue 623260 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

[Accessibility] MediaRouterElementsBrowserTest has failure for AX_ARIA_08 audit.

Project Member Reported by taku...@chromium.org, Jun 25 2016

Issue description

Error: AX_ARIA_08 (Elements with ARIA roles must ensure required owned elements are present):
#search-results

See https://github.com/GoogleChrome/accessibility-developer-tools/wiki/Audit-Rules#ax_aria_08 for more information.
 
Cc: -taku...@chromium.org
Owner: taku...@chromium.org
Is this a flaky failure? We should be catching this in the bots.

Since you've set this as "ExternalDependency" -- is there a bug tracking this in another repo? If so, please link to it here so we can keep track.

Previously, we handled the error by manually setting roles (after some discussion w/a11y and Polymer folks): 591552.
Status: Started (was: ExternalDependency)

Comment 3 by amp@chromium.org, Jun 27 2016

Labels: Hotlist-Fixit-PE2016
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 27 2016

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

commit 1a7ae6b085e411ec9be7169fce3c5fe9db2b6c76
Author: takumif <takumif@chromium.org>
Date: Mon Jun 27 20:09:17 2016

Add role="presentation" to Media Router #search-results element

To address the ARIA_08 accessibility issue [1], we added
the attribute role="presentation" to certain HTML elements [2].
However, the attribute was removed from one of the elements in
a later commit [3], so this CL adds the attribute back.

This issue surfaced while I was writing a test for the search result
list view [4].

[1] http://crbug/591552
[2] https://codereview.chromium.org/1849283004
[3] https://codereview.chromium.org/1754713005
[4] https://codereview.chromium.org/2096733002

BUG= 623260 
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2100833003
Cr-Commit-Position: refs/heads/master@{#402266}

[modify] https://crrev.com/1a7ae6b085e411ec9be7169fce3c5fe9db2b6c76/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.html

Status: Verified (was: Started)

Sign in to add a comment