Changeset 35245 in webkit for trunk/JavaScriptCore/kjs/JSNotAnObject.h

Timestamp:

Jul 18, 2008, 6:44:24 PM (17 years ago)

Author:

[email protected]

Message:

Bug 18774: SQUIRRELFISH: print meaningful error messages <​https://bugs.webkit.org/show_bug.cgi?id=18774>
<rdar://problem/5769353> SQUIRRELFISH: JavaScript error messages are missing informative text

Reviewed by Cameron Zwarich

Add support for decent error messages in JavaScript. This patch achieves this by providing
ensuring the common errors and exceptions have messages that provide the text of expression
that trigger the exception. In addition it attaches a number of properties to the exception
object detailing where in the source the expression came from.

File:

• trunk/JavaScriptCore/kjs/JSNotAnObject.h (modified) (3 diffs)

trunk/JavaScriptCore/kjs/JSNotAnObject.h

@@ -34 +34 @@
 namespace KJS {
 
+    class JSNotAnObjectErrorStub : public JSObject {
+    public:
+        JSNotAnObjectErrorStub(bool isNull)
+            : m_isNull(isNull)
+        {
+        }
+        bool isNull() const { return m_isNull; }
+        bool isNotAnObjectErrorStub() const { return true; }
+    private:
+        bool m_isNull;
+    };
+    
     // This unholy class is used to allow us to avoid multiple exception checks
     // in certain SquirrelFish opcodes -- effectively it just silently consumes
@@ -39 +51 @@
     class JSNotAnObject : public JSObject {
     public:
-        JSNotAnObject(JSObject* exception)
+        JSNotAnObject(JSNotAnObjectErrorStub* exception)
             : m_exception(exception)
         {
@@ -68 +80 @@
         virtual void getPropertyNames(ExecState*, PropertyNameArray&);
 
-        JSObject* m_exception;
+        JSNotAnObjectErrorStub* m_exception;
     };