[DevTools] Show ARIA pane based on whether AXNode is DOM node

third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilitySidebarView.js

Index: third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilitySidebarView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilitySidebarView.js b/third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilitySidebarView.js
index b3c30aef6c7a20a625627c599235a2760b9510eb..6dc7155421c87b2c57df28a6e06d0c315190c464 100644
--- a/third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilitySidebarView.js
+++ b/third_party/WebKit/Source/devtools/front_end/accessibility/AccessibilitySidebarView.js
@@ -52,10 +52,10 @@ Accessibility.AccessibilitySidebarView = class extends UI.ThrottledWidget {
 
     this._axNode = axNode;
 
-    if (axNode.ignored())
-      this._sidebarPaneStack.removeView(this._ariaSubPane);
-    else
+    if (axNode.isDOMNode())

[lushnikov, 2017/06/28 18:21:00]

What is "ignored" axNode? Should the condition be

if (axNode.isDOMNode() && !axNode.ignored())
  ...

[aboxhall, 2017/06/28 21:06:32]

I deliberately removed that. Ignored means there is no AXNode, but ARIA can
affect that so we should show the ARIA pane anyway.

       this._sidebarPaneStack.showView(this._ariaSubPane, this._axNodeSubPane);
+    else
+      this._sidebarPaneStack.removeView(this._ariaSubPane);
 
     if (this._axNodeSubPane)
       this._axNodeSubPane.setAXNode(axNode);