Prune invisible views from the desktop automation tree

ui/views/view.cc

Index: ui/views/view.cc
diff --git a/ui/views/view.cc b/ui/views/view.cc
index 95f5c7037ab53e13e4915852e1b618547a1fc345..35c1b471eac28ca4591281019a89623c5b4fd809 100644
--- a/ui/views/view.cc
+++ b/ui/views/view.cc
@@ -409,8 +409,10 @@ void View::SetVisible(bool visible) {
     AdvanceFocusIfNecessary();
 
     // Notify the parent.
-    if (parent_)
+    if (parent_) {
       parent_->ChildVisibilityChanged(this);
+      parent_->NotifyAccessibilityEvent(ui::AX_EVENT_CHILDREN_CHANGED, false);

[sky, 2016/02/10 16:19:41]

I don't see any documentation for the AXEvent constants. Is CHILDREN_CHANGED
used for changes to visibility? It would be good if there was a comment
indicating this.

+    }
 
     // This notifies all sub-views recursively.
     PropagateVisibilityNotifications(this, visible_);