Fix IsExpanded check in TreeView::PopulateAccessibilityData

Updates PopulateAccessibilityData() in |TreeView| to ensure it does not lookup row/depth information for nodes inside collapsed subtrees via GetRowForInternalNode(). Previously, the check only worked when the node's immediate parent was collapsed. This changelist expands it to examine ancestors as well.

PopulateAccessibilityData() can happen when assistive technologies attempt to retrieve property information for arbitrary tree items inside |TreeView| subtrees. Previously, this would lead to a DCHECK failure during row/depth lookup at GetRowForInternalNode().

Bug: 811277
Change-Id: I83e1a127adfcf24dac7a3a9643a659b42a198cff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2146134
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Nektarios Paisios <nektar@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Auto-Submit: Brian Liu Xu <brx@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#758843}

ui/views/controls/tree/tree_view.cc

@@ -895,7 +895,7 @@ void TreeView::PopulateAccessibilityData(InternalNode* node,
 
   } else {
     // !IsRoot(node)) && node->parent() != nullptr.
-    if (node->parent()->is_expanded()) {
+    if (IsExpanded(node->parent()->model_node())) {
       int depth = 0;
       row = GetRowForInternalNode(node, &depth);
       if (depth >= 0) {