Changeset 9526 in webkit for trunk/JavaScriptCore/kjs/object.cpp

Timestamp:

Jun 29, 2005, 1:59:21 PM (20 years ago)

Author:

ggaren

Message:

JavaScriptCore:

Patch contributed by Oliver Hunt <​[email protected]>

-fixed ​http://bugzilla.opendarwin.org/show_bug.cgi?id=3743
Incorrect error message given for certain calls

See WebCore Changelog for layout test added.

Reviewed by mjs.

• kjs/object.cpp: (KJS::ObjectImp::defaultValue):

WebCore:

-added test case for fix to ​http://bugzilla.opendarwin.org/show_bug.cgi?id=3743
Incorrect error message given for certain calls

See JavaScriptCore Changelog for details about the patch.

Reviewed by me.

Test cases added:

• layout-tests/fast/js/toString-stack-overflow-expected.txt: Added.
• layout-tests/fast/js/toString-stack-overflow.html: Added.

File:

• trunk/JavaScriptCore/kjs/object.cpp (modified) (1 diff)

trunk/JavaScriptCore/kjs/object.cpp

@@ -386 +386 @@
   }
 
+  if (exec->hadException())
+    return exec->exception();
+
   Object err = Error::create(exec, TypeError, I18N_NOOP("No default value"));
   exec->setException(err);