Modified DialogDelegateView to be accessible

chrome/browser/chromeos/options/network_config_view.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 "chrome/browser/chromeos/options/network_config_view.h"
 
 #include <algorithm>
 
 #include "ash/shell.h"
 #include "base/strings/string_util.h"
@@ skipping 192 matching lines @@
 base::string16 NetworkConfigView::GetWindowTitle() const {
   DCHECK(!child_config_view_->GetTitle().empty());
   return child_config_view_->GetTitle();
 }
 
 ui::ModalType NetworkConfigView::GetModalType() const {
   return ui::MODAL_TYPE_SYSTEM;
 }
 
 void NetworkConfigView::GetAccessibleState(ui::AXViewState* state) {
+  views::DialogDelegateView::GetAccessibleState(state);
   state->name =
       l10n_util::GetStringUTF16(IDS_OPTIONS_SETTINGS_OTHER_WIFI_NETWORKS);
-  state->role = ui::AX_ROLE_DIALOG;
 }
 
 void NetworkConfigView::ButtonPressed(views::Button* sender,
                                       const ui::Event& event) {
   if (advanced_button_ && sender == advanced_button_) {
     advanced_button_->SetVisible(false);
     ShowAdvancedView();
   }
 }
 
@@ skipping 33 matching lines @@
   result.set_height(size.height());
   if (size.width() > result.width())
     result.set_width(size.width());
   return result;
 }
 
 void NetworkConfigView::ViewHierarchyChanged(
     const ViewHierarchyChangedDetails& details) {
   // Can't init before we're inserted into a Container, because we require
   // a HWND to parent native child controls to.
+  views::DialogDelegateView::ViewHierarchyChanged(details);
   if (details.is_add && details.child == this) {
     AddChildView(child_config_view_);
   }
 }
 
 void NetworkConfigView::ShowDialog(gfx::NativeWindow parent) {
   if (parent == NULL)
     parent = GetParentForUnhostedDialog();
   // Failed connections may result in a pop-up with no natural parent window,
   // so provide a fallback context on the primary display. This is necessary
@@ skipping 71 matching lines @@
   image_view_ = new views::ImageView();
   // Disable |image_view_| so mouse events propagate to the parent.
   image_view_->SetEnabled(false);
   image_view_->SetImage(image_);
   image_view_->SetTooltipText(
       l10n_util::GetStringUTF16(IDS_OPTIONS_CONTROLLED_SETTING_POLICY));
   AddChildView(image_view_);
 }
 
 }  // namespace chromeos