Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(180)

Issue 9815027: Give the system tray an accessible role and name so that something (Closed)

Created:
8 years, 9 months ago by dmazzoni
Modified:
8 years, 9 months ago
CC:
chromium-reviews, hashimoto+watch_chromium.org, ben+watch_chromium.org, aboxhall+watch_chromium.org, dhollowa+watch_chromium.org, yoshiki+watch_chromium.org, yuzo+watch_chromium.org, davidbarr+watch_chromium.org, dmazzoni+watch_chromium.org, dtseng+watch_chromium.org, ctguil+watch_chromium.org, zork+watch_chromium.org
Visibility:
Public.

Description

Give the system tray an accessible role and name so that something appropriate is announced when it's focused. BUG=119238 TEST=enable accessibility, click on system tray Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=128173

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Patch
M ash/ash_strings.grd View 1 chunk +3 lines, -0 lines 0 comments Download
M ash/system/tray/system_tray.cc View 2 chunks +10 lines, -0 lines 0 comments Download
M chrome/browser/ui/views/accessibility_event_router_views.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
dmazzoni
Let me know if there's already a string for "status tray" or if you wanted ...
8 years, 9 months ago (2012-03-21 21:47:17 UTC) #1
sadrul
On 2012/03/21 21:47:17, Dominic Mazzoni wrote: > Let me know if there's already a string ...
8 years, 9 months ago (2012-03-22 00:38:19 UTC) #2
dmazzoni
+ben for OWNERS review
8 years, 9 months ago (2012-03-22 02:17:49 UTC) #3
Ben Goodger (Google)
lgtm
8 years, 9 months ago (2012-03-22 02:39:45 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dmazzoni@chromium.org/9815027/1
8 years, 9 months ago (2012-03-22 02:44:57 UTC) #5
commit-bot: I haz the power
8 years, 9 months ago (2012-03-22 04:54:01 UTC) #6
Change committed as 128173

Powered by Google App Engine
This is Rietveld 408576698