Minor improvements to views accessibility. Improves notification for a

few classes, and replaces an ugly view class name comparison with a clean
switch based on the view's role.

BUG=none
TEST=none

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

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@91913 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 9 insertions, 0 deletions

diff --git a/views/controls/message_box_view.cc b/views/controls/message_box_view.cc
index df35312..2710894 100644
--- a/views/controls/message_box_view.cc
+++ b/views/controls/message_box_view.cc
@@ -7,6 +7,7 @@
 #include "base/i18n/rtl.h"
 #include "base/message_loop.h"
 #include "base/utf_string_conversions.h"
+#include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/clipboard/clipboard.h"
 #include "ui/base/clipboard/scoped_clipboard_writer.h"
 #include "ui/base/message_box_flags.h"
@@ -17,6 +18,7 @@
 #include "views/layout/grid_layout.h"
 #include "views/layout/layout_constants.h"
 #include "views/views_delegate.h"
+#include "views/widget/widget.h"
 #include "views/window/client_view.h"
 
 static const int kDefaultMessageWidth = 320;
@@ -85,6 +87,10 @@ void MessageBoxView::SetCheckBoxSelected(bool selected) {
   checkbox_->SetChecked(selected);
 }
 
+void MessageBoxView::GetAccessibleState(ui::AccessibleViewState* state) {
+  state->role = ui::AccessibilityTypes::ROLE_ALERT;
+}
+
 ///////////////////////////////////////////////////////////////////////////////
 // MessageBoxView, View overrides:
 
@@ -94,6 +100,9 @@ void MessageBoxView::ViewHierarchyChanged(bool is_add,
   if (child == this && is_add) {
     if (prompt_field_)
       prompt_field_->SelectAll();
+
+    GetWidget()->NotifyAccessibilityEvent(
+        this, ui::AccessibilityTypes::EVENT_ALERT, true);
   }
 }