New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 614922 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows , Chrome , Mac
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Playing audio icon in tab not accessible to screen readers

Project Member Reported by lpalmaro@chromium.org, May 26 2016

Issue description

Version: 50.0.2661.102
OS: Mac, Win, Chrome OS 

What steps will reproduce the problem?
(1) Open a tab in Chrome and navigate to youtube.com, and play a video with audio. Open at least one other tab. 
(2) On Mac (for example, turn on the VoiceOver screen reader by pressing command F5. 
(3) Press command option right arrow to navigate through tabs using the keyboard, and listen to the spoken feedback. 

on Chrome OS:
1) Open a tab in Chrome and navigate to youtube.com, and play a video with audio. Open at least one other tab. 
2) Enable ChromeVox by pressing ctrl alt z
3) Press ctrl tab to navigate through tabs using the keyboard, and listen to the spoken feedback. 

What is the expected output?
The given screen reader should read the tab title when switching tabs. But in instances where an audio icon appears to tell the user audio is playing in that tab, Voiceover (or the given screen reader) should also read a label that speaks "currently playing audio", or something along those lines. 

What do you see instead?
There is no spoken indication that this icon appears. Let's work together to determine the best experience for screen reader users, without adding too much verbosity. 


Please use labels and text to provide additional information.

 

Comment 1 by rolfe@chromium.org, May 27 2016

Cc: renganat...@chromium.org
(+ renganathan)

renganathan@ - would you be the best PM to find a SWE for this media accessibility issue? Who owns the audio tab indicator (is it security instead of the media team?)

Comment 2 by chaok@google.com, Dec 5 2016

Labels: -Pri-3 Pri-1
For views, tab status audio descriptions were added in this change as part of https://bugs.chromium.org/p/chromium/issues/detail?id=635610:
 
https://chromium.googlesource.com/chromium/src.git/+/bfeeddcdf2925a4fcd7bcf9666c8af703fa35e29

We'll need someone to implement the equivalent on Mac. 
Labels: NewComponent-Accessibility NewComponent-Accessibility-ChromeVox
Labels: -NewComponent-Accessibility-ChromeVox NewComponent-Accessibility-Browser

Comment 6 by rolfe@chromium.org, Mar 30 2017

Cc: -rolfe@chromium.org
Removing myself as contact. Might be an issue for rachelis@ in media-land.
Labels: -newcomponent-accessibility-browser -newcomponent-accessibility
Labels: -Pri-1 Pri-2
NextAction: 2017-08-18
Owner: ellyjo...@chromium.org
Status: Assigned (was: Available)
I'm just gonna stick "muted" on the end of muted tabs (or "playing audio")
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 7 2017

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

commit b27df3cbe3d78732be49699a07f7ea9a01eb0c35
Author: Elly Fong-Jones <ellyjones@chromium.org>
Date: Mon Aug 07 18:11:36 2017

cocoa: include tab alert state & crash state in accessible title

This change:
1) Factors out most of BrowserView::GetAccessibleTabLabel into a new helper,
   chrome::AssembleTabAccessibilityLabel
2) Has BrowserView::GetAccessibleTabLabel call the new helper
3) Adds [TabController accessibleTitle] to the Cocoa TabController
4) Has the Cocoa TabView use [TabController accessibleTitle] for the
   accessible title
5) Has [TabController accessibleTitle] call the new helper

Bug:  614922 
Change-Id: I94790fea08ba173b893a872fedc278ae5c8b8434
Reviewed-on: https://chromium-review.googlesource.com/602807
Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#492362}
[modify] https://crrev.com/b27df3cbe3d78732be49699a07f7ea9a01eb0c35/chrome/browser/ui/cocoa/tabs/tab_controller.h
[modify] https://crrev.com/b27df3cbe3d78732be49699a07f7ea9a01eb0c35/chrome/browser/ui/cocoa/tabs/tab_controller.mm
[modify] https://crrev.com/b27df3cbe3d78732be49699a07f7ea9a01eb0c35/chrome/browser/ui/cocoa/tabs/tab_view.mm
[modify] https://crrev.com/b27df3cbe3d78732be49699a07f7ea9a01eb0c35/chrome/browser/ui/tabs/tab_utils.cc
[modify] https://crrev.com/b27df3cbe3d78732be49699a07f7ea9a01eb0c35/chrome/browser/ui/tabs/tab_utils.h
[modify] https://crrev.com/b27df3cbe3d78732be49699a07f7ea9a01eb0c35/chrome/browser/ui/views/frame/browser_view.cc

Status: Fixed (was: Assigned)
The NextAction date has arrived: 2017-08-18
NextAction: ----

Sign in to add a comment