New issue
Advanced search Search tips

Issue 671094 link

Starred by 4 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Stop Casting button in status bar should specify $media $device

Project Member Reported by chaok@google.com, Dec 5 2016

Issue description

Mode: force_next
Version: 57.0.2940.0
Reproduction Steps: 
1. Cast $media to $device
2. Alt+Shift+S for status tray > TAB to Stop button
3. Search+Left Arrow will utter "Casting $media to $device"

Observed: Stop button
Expected: Stop Casting $media to $device
 
Owner: ----
Please ask the author of this view to fix.

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

@Laura: could you help find owner?
Thanks!

Comment 3 by dtseng@chromium.org, Jan 12 2017

Cc: jdufault@chromium.org
One way of finding an owner is to see who's touched something similar in the codebase. For example, searching on crbug, I found:
https://bugs.chromium.org/p/chromium/issues/detail?id=489445
I would try the owner of that bug.

+ cc jdufault@chromium.org
for any help with this. Thanks!
Owner: jdufault@chromium.org
Status: Assigned (was: Untriaged)
I'll try to take a look soon. Should be a relatively simple fix.
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Jan 30 2017

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

commit 4e949b4732c4d76670a39719a73422ed0dc2976c
Author: jdufault <jdufault@chromium.org>
Date: Mon Jan 30 20:07:58 2017

cros: Fix various small issues with cast tray entry.

673023: Do not show TrayCast active state until we have media router
information to prevent short visual glitch where we are "Casting to
unknown receiver".

683908: Only show local routes

671094: Better accessibility label for Stop casting button.

BUG=673023, 683908 , 671094 

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

[modify] https://crrev.com/4e949b4732c4d76670a39719a73422ed0dc2976c/ash/ash_strings.grd
[modify] https://crrev.com/4e949b4732c4d76670a39719a73422ed0dc2976c/ash/common/system/chromeos/cast/tray_cast.cc
[modify] https://crrev.com/4e949b4732c4d76670a39719a73422ed0dc2976c/ash/common/system/chromeos/screen_security/screen_tray_item.h
[modify] https://crrev.com/4e949b4732c4d76670a39719a73422ed0dc2976c/chrome/browser/ui/ash/system_tray_tray_cast_browsertest_media_router_chromeos.cc

Status: Fixed (was: Started)
Project Member

Comment 8 by bugdroid1@chromium.org, Feb 16 2017

Labels: merge-merged-2987
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1dbca8ec13dcf78b1c689018527315856c2b0a95

commit 1dbca8ec13dcf78b1c689018527315856c2b0a95
Author: Jacob Dufault <jdufault@google.com>
Date: Thu Feb 16 19:13:02 2017

cros: Fix various small issues with cast tray entry.

673023: Do not show TrayCast active state until we have media router
information to prevent short visual glitch where we are "Casting to
unknown receiver".

683908: Only show local routes

671094: Better accessibility label for Stop casting button.

BUG=673023, 683908 , 671094 

Review-Url: https://codereview.chromium.org/2656003006
Cr-Commit-Position: refs/heads/master@{#447045}
(cherry picked from commit 461840ed16b4e886966d5bc44558668f8df382b7)

Review-Url: https://codereview.chromium.org/2697363002 .
Cr-Commit-Position: refs/branch-heads/2987@{#553}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/1dbca8ec13dcf78b1c689018527315856c2b0a95/ash/ash_strings.grd
[modify] https://crrev.com/1dbca8ec13dcf78b1c689018527315856c2b0a95/ash/common/system/chromeos/cast/tray_cast.cc
[modify] https://crrev.com/1dbca8ec13dcf78b1c689018527315856c2b0a95/ash/common/system/chromeos/screen_security/screen_tray_item.h
[modify] https://crrev.com/1dbca8ec13dcf78b1c689018527315856c2b0a95/chrome/browser/ui/ash/system_tray_tray_cast_browsertest_media_router_chromeos.cc

Comment 9 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 10 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Status: Verified (was: Fixed)

Sign in to add a comment