Commit 47d8631b authored by dmazzoni@chromium.org's avatar dmazzoni@chromium.org

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


Review URL: http://codereview.chromium.org/9815027

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@128173 0039d316-1c4b-4281-b951-d872f2087c98
parent 62b23c2f
...@@ -175,6 +175,9 @@ This file contains the strings for ash. ...@@ -175,6 +175,9 @@ This file contains the strings for ash.
</message> </message>
<!-- Status tray items --> <!-- Status tray items -->
<message name="IDS_ASH_STATUS_TRAY_ACCESSIBLE_NAME" desc="The accessible name of the status tray.">
Status tray
</message>
<message name="IDS_ASH_STATUS_TRAY_SETTINGS_AND_HELP" desc="The label used for the help-and-settings item in the status tray."> <message name="IDS_ASH_STATUS_TRAY_SETTINGS_AND_HELP" desc="The label used for the help-and-settings item in the status tray.">
Settings and Help Settings and Help
</message> </message>
......
...@@ -17,10 +17,13 @@ ...@@ -17,10 +17,13 @@
#include "base/logging.h" #include "base/logging.h"
#include "base/timer.h" #include "base/timer.h"
#include "base/utf_string_conversions.h" #include "base/utf_string_conversions.h"
#include "grit/ash_strings.h"
#include "third_party/skia/include/core/SkCanvas.h" #include "third_party/skia/include/core/SkCanvas.h"
#include "third_party/skia/include/core/SkColor.h" #include "third_party/skia/include/core/SkColor.h"
#include "third_party/skia/include/core/SkPaint.h" #include "third_party/skia/include/core/SkPaint.h"
#include "third_party/skia/include/core/SkPath.h" #include "third_party/skia/include/core/SkPath.h"
#include "ui/base/accessibility/accessible_view_state.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/gfx/canvas.h" #include "ui/gfx/canvas.h"
#include "ui/gfx/skia_util.h" #include "ui/gfx/skia_util.h"
#include "ui/views/border.h" #include "ui/views/border.h"
...@@ -335,6 +338,13 @@ class SystemTrayBubble : public views::BubbleDelegateView { ...@@ -335,6 +338,13 @@ class SystemTrayBubble : public views::BubbleDelegateView {
} }
} }
// Overridden from views::View.
virtual void GetAccessibleState(ui::AccessibleViewState* state) OVERRIDE {
state->role = ui::AccessibilityTypes::ROLE_WINDOW;
state->name = l10n_util::GetStringUTF16(
IDS_ASH_STATUS_TRAY_ACCESSIBLE_NAME);
}
virtual bool CanActivate() const OVERRIDE { virtual bool CanActivate() const OVERRIDE {
return can_activate_; return can_activate_;
} }
......
...@@ -135,6 +135,7 @@ void AccessibilityEventRouterViews::DispatchAccessibilityNotification( ...@@ -135,6 +135,7 @@ void AccessibilityEventRouterViews::DispatchAccessibilityNotification(
view->GetAccessibleState(&state); view->GetAccessibleState(&state);
switch (state.role) { switch (state.role) {
case ui::AccessibilityTypes::ROLE_ALERT: case ui::AccessibilityTypes::ROLE_ALERT:
case ui::AccessibilityTypes::ROLE_WINDOW:
SendWindowNotification(view, type, profile); SendWindowNotification(view, type, profile);
break; break;
case ui::AccessibilityTypes::ROLE_BUTTONMENU: case ui::AccessibilityTypes::ROLE_BUTTONMENU:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment