677808 - Implement uncaughtExceptionHook spec behavior when the hook throws an exception

Status: NEW

(Core :: JavaScript Engine, task, P5)

Description

[Jason Orendorff [:jorendorff]]

If a debugging hook throws an exception and the uncaughtExceptionHook is null, or it also throws an error, we don't do what the spec says:

https://wiki.mozilla.org/Debugger#Accessor_Properties_of_the_Debugger_Prototype_Object

Comment 1

[Bryan Thrall [:bthrall]]

We do have Debugger::reportUncaughtException(), but it doesn't look like it is called everywhere that Debugger::callUncaughtExceptionHandler() is called.

The behavior in the spec that appears to be missing is:

If the uncaught exception hook itself throws an exception, uncaught-hook-exception, SpiderMonkey throws a new error object, confess-to-debuggee-exception, to the debuggee whose message blames the debugger, and includes textual descriptions of uncaught-hook-exception and the original debugger-exception.

If uncaughtExceptionHook’s value is null, SpiderMonkey throws an exception to the debuggee whose message blames the debugger, and includes a textual description of debugger-exception.