Web Inspector: add more context to Object.addEventListener assertions
authorbburg@apple.com <bburg@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Jan 2014 21:52:44 +0000 (21:52 +0000)
committerbburg@apple.com <bburg@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 16 Jan 2014 21:52:44 +0000 (21:52 +0000)
https://bugs.webkit.org/show_bug.cgi?id=127125

Reviewed by Joseph Pecoraro.

Log invalid event type and listener parameters, and log the other
parameters so the callsite is easier to figure out.

* UserInterface/Object.js:
(WebInspector.Object.addEventListener):

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

Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Object.js

index f6f64ac..4e22fa4 100644 (file)
@@ -1,5 +1,18 @@
 2014-01-16  Brian Burg  <bburg@apple.com>
 
+        Web Inspector: add more context to Object.addEventListener assertions
+        https://bugs.webkit.org/show_bug.cgi?id=127125
+
+        Reviewed by Joseph Pecoraro.
+
+        Log invalid event type and listener parameters, and log the other
+        parameters so the callsite is easier to figure out.
+
+        * UserInterface/Object.js:
+        (WebInspector.Object.addEventListener):
+
+2014-01-16  Brian Burg  <bburg@apple.com>
+
         Web Inspector: CallFrame is missing saveIdentityToCookie, or should be excluded
         https://bugs.webkit.org/show_bug.cgi?id=126791
 
index 2c3ab19..0321875 100644 (file)
@@ -55,11 +55,11 @@ WebInspector.Object.addEventListener = function(eventType, listener, thisObject)
 {
     thisObject = thisObject || null;
 
-    console.assert(eventType);
+    console.assert(eventType, "Object.addEventListener: invalid event type ", eventType, "(listener: ", listener, "thisObject: ", thisObject, ")");
     if (!eventType)
         return;
 
-    console.assert(listener);
+    console.assert(listener, "Object.addEventListener: invalid listener ", listener, "(event type: ", eventType, "thisObject: ", thisObject, ")");
     if (!listener)
         return;