Modified DialogDelegateView to be accessible

ui/views/window/dialog_delegate.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/views/window/dialog_delegate.h"
 
 #include "base/logging.h"
+#include "ui/accessibility/ax_view_state.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/strings/grit/ui_strings.h"
 #include "ui/views/bubble/bubble_border.h"
 #include "ui/views/bubble/bubble_frame_view.h"
 #include "ui/views/controls/button/label_button.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_observer.h"
 #include "ui/views/window/dialog_client_view.h"
 
 #if defined(OS_WIN)
@@ skipping 213 matching lines @@
 }
 
 const Widget* DialogDelegateView::GetWidget() const {
   return View::GetWidget();
 }
 
 View* DialogDelegateView::GetContentsView() {
   return this;
 }
 
+void DialogDelegateView::GetAccessibleState(ui::AXViewState* state) {
+  state->name = GetDialogTitle();
+  state->role = ui::AX_ROLE_DIALOG;
+}
+
+void DialogDelegateView::ViewHierarchyChanged(
+    const ViewHierarchyChangedDetails& details) {
+  if (details.is_add && details.child == this)

[sky, 2014/10/31 02:33:05]

I think you should verify you're in a widget here too, eg GetWidget() returns
non-null.

[sashab, 2014/10/31 02:56:47]

Good call. Added to the if-statement.

+    NotifyAccessibilityEvent(ui::AX_EVENT_ALERT, true);
+}
+
 }  // namespace views