Changeset 147872 in webkit for trunk/Source/JavaScriptCore/ChangeLog

Timestamp:

Apr 7, 2013, 11:47:20 AM (12 years ago)

Author:

[email protected]

Message:

Inspector should display information about non-object exceptions
​https://bugs.webkit.org/show_bug.cgi?id=114123

Reviewed by Adele Peterson.

Source/JavaScriptCore:

Make sure we store the right stack information, even when throwing
a primitive.

• interpreter/CallFrame.h:

(JSC::ExecState::clearSupplementaryExceptionInfo):
(ExecState):

• interpreter/Interpreter.cpp:

(JSC::Interpreter::addStackTraceIfNecessary):
(JSC::Interpreter::throwException):

Source/WebCore:

Make use of the stack trace for line information when we're reporting
an exception

• bindings/js/JSDOMBinding.cpp:

(WebCore::reportException):

LayoutTests:

All these tests throw primitives as exceptions, and now they have source
and line number information

• fast/dom/exception-getting-event-handler-expected.txt:
• fast/dom/javascript-url-exception-isolation-expected.txt:
• fast/dom/nested-script-exceptions-expected.txt:
• fast/events/onerror-no-constructor-expected.txt:
• fast/events/window-onerror13-expected.txt:
• fast/events/window-onerror16-expected.txt:
• fast/events/window-onerror2-expected.txt:
• fast/events/window-onerror8-expected.txt:
• fast/js/uncaught-exception-line-number-expected.txt:
• fast/sub-pixel/inline-block-with-padding-expected.txt:
• platform/mac/fast/AppleScript/001-expected.txt:

File:

• trunk/Source/JavaScriptCore/ChangeLog (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2013-04-07  Oliver Hunt  <[email protected]>
+
+        Inspector should display information about non-object exceptions
+        https://bugs.webkit.org/show_bug.cgi?id=114123
+
+        Reviewed by Adele Peterson.
+
+        Make sure we store the right stack information, even when throwing
+        a primitive.
+
+        * interpreter/CallFrame.h:
+        (JSC::ExecState::clearSupplementaryExceptionInfo):
+        (ExecState):
+        * interpreter/Interpreter.cpp:
+        (JSC::Interpreter::addStackTraceIfNecessary):
+        (JSC::Interpreter::throwException):
+
 2013-04-06  Oliver Hunt  <[email protected]>