Web Inspector: Type Token View shows type information on hover when it shouldn't
authorsaambarati1@gmail.com <saambarati1@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Sep 2014 21:39:53 +0000 (21:39 +0000)
committersaambarati1@gmail.com <saambarati1@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 26 Sep 2014 21:39:53 +0000 (21:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=137152

Reviewed by Brian J. Burg.

TypeTokenView should only show type information when there is
more than one primitive type or more than zero objects types.
This fixes a bug in checking the length of object types.

* UserInterface/Views/TypeTokenView.js:
(WebInspector.TypeTokenView.prototype._shouldShowPopover):

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

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

index 7e2e6c6937c59fbbfb15ddeb9f167d30d3f71099..621b7d4f24ca4ffbb4cc6c11fd6f975fd20b5fa7 100644 (file)
@@ -1,3 +1,17 @@
+2014-09-26  Saam Barati  <saambarati1@gmail.com>
+
+        Web Inspector: Type Token View shows type information on hover when it shouldn't
+        https://bugs.webkit.org/show_bug.cgi?id=137152
+
+        Reviewed by Brian J. Burg.
+
+        TypeTokenView should only show type information when there is 
+        more than one primitive type or more than zero objects types. 
+        This fixes a bug in checking the length of object types.
+
+        * UserInterface/Views/TypeTokenView.js:
+        (WebInspector.TypeTokenView.prototype._shouldShowPopover):
+
 2014-09-25  Timothy Hatcher  <timothy@apple.com>
 
         Web Inspector: Show type info icon is blurry
index cf25c817c7632ee40901d91fabffaf7f41412765..0351a1b9f6814c57d4df315f4ee61eefe432334c 100644 (file)
@@ -132,7 +132,7 @@ WebInspector.TypeTokenView.prototype = {
         if (this._types.primitiveTypeNames && this._types.primitiveTypeNames.length > 1)
             return true;
 
-        if (this._types.structures && this._types.structures)
+        if (this._types.structures && this._types.structures.length)
             return true;
 
         return false;