Web Inspector: Always expand initial top level DOM tree nodes when not including the root (frame views)

JosephPecoraro · webkit-commit-queue · commit c3c28fa74045 · 2015-05-05T01:00:49.000Z
https://bugs.webkit.org/show_bug.cgi?id=144607 Patch by Joseph Pecoraro <pecoraro@apple.com> on 2015-05-04 Reviewed by Timothy Hatcher. * UserInterface/Views/DOMTreeOutline.js: (WebInspector.DOMTreeOutline.prototype.update): Canonical link: https://commits.webkit.org/162621@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@183786 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/Source/WebInspectorUI/ChangeLog b/Source/WebInspectorUI/ChangeLog
@@ -1,3 +1,13 @@
+2015-05-04  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Web Inspector: Always expand initial top level DOM tree nodes when not including the root (frame views)
+        https://bugs.webkit.org/show_bug.cgi?id=144607
+
+        Reviewed by Timothy Hatcher.
+
+        * UserInterface/Views/DOMTreeOutline.js:
+        (WebInspector.DOMTreeOutline.prototype.update):
+
 2015-05-04  Timothy Hatcher  <timothy@apple.com>
 
         Web Inspector: Allow closing and reopening the Debugger tab
diff --git a/Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js b/Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js
@@ -174,6 +174,9 @@ WebInspector.DOMTreeOutline = class DOMTreeOutline extends WebInspector.TreeOutl
                 treeElement.selectable = this._selectEnabled;
                 this.appendChild(treeElement);
                 node = node.nextSibling;
+                
+                if (treeElement.hasChildren && !treeElement.expanded)
+                    treeElement.expand();
             }
         }
 

Original file line number	Diff line number	Diff line change
`@@ -174,6 +174,9 @@ WebInspector.DOMTreeOutline = class DOMTreeOutline extends WebInspector.TreeOutl`
`174`	`174`	`treeElement.selectable = this._selectEnabled;`
`175`	`175`	`this.appendChild(treeElement);`
`176`	`176`	`node = node.nextSibling;`
	`177`	`+`
	`178`	`+ if (treeElement.hasChildren && !treeElement.expanded)`
	`179`	`+ treeElement.expand();`
`177`	`180`	`}`
`178`	`181`	`}`
`179`	`182`