GetFocusedChildWidgetForAccessibility should return any focused widget

Modify ViewAccessibilityUtils::GetFocusedChildWidgetForAccessibility to
return any widget that is visible and focused.

Previously, this was limited to tab-modal dialogs. Other focused
widgets, like the Find bar, should be treated in the same way.

Bug: 869703
Change-Id: I720bdd1e9c8d665d839296aa9bfd4c9490805dbd
Reviewed-on: https://chromium-review.googlesource.com/1208810Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: James Wallace-Lee <jamwalla@chromium.org>
Cr-Commit-Position: refs/heads/master@{#590523}

ui/views/accessibility/view_accessibility_utils.cc

@@ -37,8 +37,7 @@ Widget* ViewAccessibilityUtils::GetFocusedChildWidgetForAccessibility(
 // static
 bool ViewAccessibilityUtils::IsFocusedChildWidget(Widget* widget,
                                                   const View* focused_view) {
-  return widget->widget_delegate()->GetModalType() == ui::MODAL_TYPE_CHILD &&
-         widget->IsVisible() &&
+  return widget->IsVisible() &&
          widget->GetContentsView()->Contains(focused_view);
 };
 

ui/views/accessibility/view_accessibility_utils.h

@@ -20,8 +20,7 @@ class VIEWS_EXPORT ViewAccessibilityUtils {
   static Widget* GetFocusedChildWidgetForAccessibility(const View* view);
 
   // Used by GetFocusedChildWidgetForAccessibility to determine if a Widget
-  // should be handled separately. When visible and focused, these include:
-  //  - tab modal dialogs
+  // should be handled separately.
   static bool IsFocusedChildWidget(Widget* widget, const View* focused_view);
 };