summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authormaruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-09-23 21:00:39 +0000
committermaruel@chromium.org <maruel@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2010-09-23 21:00:39 +0000
commit943ca53d131a04f71e18fd140780b39304492ec1 (patch)
treed2b2c3596591fcfacbfe3cef00118cb881e46d73
parent3c06fe97418acda683869927732eb3cf66611443 (diff)
downloadchromium_src-943ca53d131a04f71e18fd140780b39304492ec1.zip
chromium_src-943ca53d131a04f71e18fd140780b39304492ec1.tar.gz
chromium_src-943ca53d131a04f71e18fd140780b39304492ec1.tar.bz2
Revert "Set focus to a ConfirmInfoBar. This serves to alerts screen reader users ..."
This reverts commit 60342. TBR=dtseng BUG=none TEST=none Review URL: http://codereview.chromium.org/3472008 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@60348 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--chrome/browser/views/infobars/infobars.cc4
-rw-r--r--views/accessibility/accessibility_types.h2
-rw-r--r--views/accessibility/view_accessibility.cc4
3 files changed, 1 insertions, 9 deletions
diff --git a/chrome/browser/views/infobars/infobars.cc b/chrome/browser/views/infobars/infobars.cc
index 1fe3921..58a4db6 100644
--- a/chrome/browser/views/infobars/infobars.cc
+++ b/chrome/browser/views/infobars/infobars.cc
@@ -127,7 +127,7 @@ InfoBar::~InfoBar() {
bool InfoBar::GetAccessibleRole(AccessibilityTypes::Role* role) {
DCHECK(role);
- *role = AccessibilityTypes::ROLE_ALERT;
+ *role = AccessibilityTypes::ROLE_PANE;
return true;
}
@@ -253,8 +253,6 @@ void InfoBar::InfoBarAdded() {
GetFocusManager()));
}
#endif
-
- NotifyAccessibilityEvent(AccessibilityTypes::EVENT_ALERT);
}
void InfoBar::InfoBarRemoved() {
diff --git a/views/accessibility/accessibility_types.h b/views/accessibility/accessibility_types.h
index b30208f..4470344 100644
--- a/views/accessibility/accessibility_types.h
+++ b/views/accessibility/accessibility_types.h
@@ -47,7 +47,6 @@ class AccessibilityTypes {
// must provide a conversion to its own roles (see e.g.
// ViewAccessibility::get_accRole and ViewAccessibility::MSAARole).
enum Role {
- ROLE_ALERT,
ROLE_APPLICATION,
ROLE_BUTTONDROPDOWN,
ROLE_BUTTONMENU,
@@ -83,7 +82,6 @@ class AccessibilityTypes {
// events must provide a conversion to its own events (see e.g.
// ViewAccessibility::MSAAEvent).
enum Event {
- EVENT_ALERT,
EVENT_FOCUS,
EVENT_MENUSTART,
EVENT_MENUEND,
diff --git a/views/accessibility/view_accessibility.cc b/views/accessibility/view_accessibility.cc
index 985097f..07eec0d 100644
--- a/views/accessibility/view_accessibility.cc
+++ b/views/accessibility/view_accessibility.cc
@@ -675,8 +675,6 @@ STDMETHODIMP ViewAccessibility::put_accValue(VARIANT var_id, BSTR put_val) {
int32 ViewAccessibility::MSAAEvent(AccessibilityTypes::Event event) {
switch (event) {
- case AccessibilityTypes::EVENT_ALERT:
- return EVENT_SYSTEM_ALERT;
case AccessibilityTypes::EVENT_FOCUS:
return EVENT_OBJECT_FOCUS;
case AccessibilityTypes::EVENT_MENUSTART:
@@ -696,8 +694,6 @@ int32 ViewAccessibility::MSAAEvent(AccessibilityTypes::Event event) {
int32 ViewAccessibility::MSAARole(AccessibilityTypes::Role role) {
switch (role) {
- case AccessibilityTypes::ROLE_ALERT:
- return ROLE_SYSTEM_ALERT;
case AccessibilityTypes::ROLE_APPLICATION:
return ROLE_SYSTEM_APPLICATION;
case AccessibilityTypes::ROLE_BUTTONDROPDOWN: