Web Inspector: TypeError when hovering over "Show All Nodes" button in the Dom Tree...
authorjonowells@apple.com <jonowells@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jun 2014 20:59:51 +0000 (20:59 +0000)
committerjonowells@apple.com <jonowells@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jun 2014 20:59:51 +0000 (20:59 +0000)
https://bugs.webkit.org/show_bug.cgi?id=133699

Reviewed by Geoffrey Garen.

The call to element._createTooltipForNode() was failing because element was sometimes not a
DOMTreeOutline but rather only a TreeElement (in the case of the "Show All Nodes" button specifically).
A check for type fixes this issue.

* UserInterface/Views/DOMTreeOutline.js:
(WebInspector.DOMTreeOutline.prototype._onmousemove): Check for presence of _createTooltipForNode.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@170389 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Views/DOMTreeOutline.js

index 757181b..b4aa813 100644 (file)
@@ -1,3 +1,17 @@
+2014-06-24  Jono Wells  <jonowells@apple.com>
+
+        Web Inspector: TypeError when hovering over "Show All Nodes" button in the Dom Tree inspector
+        https://bugs.webkit.org/show_bug.cgi?id=133699
+
+        Reviewed by Geoffrey Garen.
+
+        The call to element._createTooltipForNode() was failing because element was sometimes not a 
+        DOMTreeOutline but rather only a TreeElement (in the case of the "Show All Nodes" button specifically).
+        A check for type fixes this issue.
+
+        * UserInterface/Views/DOMTreeOutline.js:
+        (WebInspector.DOMTreeOutline.prototype._onmousemove): Check for presence of _createTooltipForNode.
+
 2014-06-21  Jono Wells  <jonowells@apple.com>
 
         Web Inspector: UI fonts should pull from system fonts automatically to whatever degree possible
index e2e9ad7..2bf2d14 100644 (file)
@@ -309,7 +309,7 @@ WebInspector.DOMTreeOutline.prototype = {
             this._previousHoveredElement = element;
 
             // Lazily compute tag-specific tooltips.
-            if (element.representedObject && !element.tooltip)
+            if (element.representedObject && !element.tooltip && element._createTooltipForNode)
                 element._createTooltipForNode();
         }