Modified DialogDelegateView to be accessible

ui/views/window/dialog_delegate.h

Index: ui/views/window/dialog_delegate.h
diff --git a/ui/views/window/dialog_delegate.h b/ui/views/window/dialog_delegate.h
index dae317ebf34457ab452722863231f1b43dab3809..2347178e7c366838c29462727a611f68fe673502 100644
--- a/ui/views/window/dialog_delegate.h
+++ b/ui/views/window/dialog_delegate.h
@@ -137,6 +137,11 @@ class VIEWS_EXPORT DialogDelegateView : public DialogDelegate,
   View* GetContentsView() override;
 
  private:
+  // Overridden from View:

[sky, 2014/10/30 13:13:28]

I don't think either of these should be private. Subclasses may want to
override. In fact you need to investigate all subclasses and make sure if one is
overriding they are invoking DialogDelegateView::Foo.

[sashab, 2014/10/30 23:12:02]

Looking at all the direct subclasses, the following override
ViewHierarchyChanged:
- NetworkConfigView
- CollectedCookieView
- ExtensionInstallDialogView
- HungRendererView
- ProfileSigninConfirmationDialogViews
- SignedCertificateTimestampsViews
- TaskManagerView
I have updated these to call the parent ViewHierarchyChanged. I don't know an
easy way to search through all their subclasses recursively... to find all
classes that override ViewHierarchyChanged that also indirectly inherit from
DialogDelegateView.

The following override GetAccessibleState:
- NetworkConfigView
- BookmarkEditorView
However, I've left them as they are since they override GetWindowTitle() to use
a different title (they want to display one title but have a different one for
accessibility purposes). Made them call the parent class just to remove the
repeated 'role = AX_ROLE_DIALOG' code.

+  void GetAccessibleState(ui::AXViewState* state) override;
+  void ViewHierarchyChanged(
+      const ViewHierarchyChangedDetails& details) override;
+
   DISALLOW_COPY_AND_ASSIGN(DialogDelegateView);
 };