[webkit-reviews] review granted: [Bug 21036] Inspecting a node after reloading the page with the Inspector open does not reveal the node : [Attachment 23716] Proposed patch

[bugzilla-daemon at webkit.org]

Kevin McCullough <kmccullough at apple.com> has granted Timothy Hatcher
<timothy at hatcher.name>'s request for review:
Bug 21036: Inspecting a node after reloading the page with the Inspector open
does not reveal the node
https://bugs.webkit.org/show_bug.cgi?id=21036

Attachment 23716: Proposed patch
https://bugs.webkit.org/attachment.cgi?id=23716&action=edit

------- Additional Comments from Kevin McCullough <kmccullough at apple.com>
-    if (child.treeOutline)
+    if (child.treeOutline) {
	 child.treeOutline._forgetTreeElement(child);
+	 child.treeOutline._forgetChildrenRecursive(child);
+    }

Should we always assume _forgetChildrenRecursive should follow
_forgetTreeElement and thus include it in _forgetTree Element?