Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(37)

Unified Diff: ui/views/accessibility/native_view_accessibility_win.cc

Issue 588113002: Change Widget::GetAllOwnedWidgets to return child widgets as well (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@aid_resilient_to_uninstall_and_profile_changes
Patch Set: Minor fix Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/accessibility/native_view_accessibility_win.cc
diff --git a/ui/views/accessibility/native_view_accessibility_win.cc b/ui/views/accessibility/native_view_accessibility_win.cc
index 023e31188e3b86e52d0cf58b05195f8c031d2b2f..c27a1cecb12d8b68345ca217eaae5a00b8f4f47a 100644
--- a/ui/views/accessibility/native_view_accessibility_win.cc
+++ b/ui/views/accessibility/native_view_accessibility_win.cc
@@ -1491,13 +1491,11 @@ void NativeViewAccessibilityWin::PopulateChildWidgetVector(
return;
std::set<Widget*> child_widgets;
- Widget::GetAllChildWidgets(widget->GetNativeView(), &child_widgets);
Widget::GetAllOwnedWidgets(widget->GetNativeView(), &child_widgets);
for (std::set<Widget*>::const_iterator iter = child_widgets.begin();
iter != child_widgets.end(); ++iter) {
Widget* child_widget = *iter;
- if (child_widget == widget)
- continue;
+ DCHECK_NE(widget, child_widget);
if (!child_widget->IsVisible())
continue;
« no previous file with comments | « chrome/browser/ui/app_list/app_list_service_views_browsertest.cc ('k') | ui/views/widget/native_widget_aura.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698