Capture full backtrace when a native function calls back into JS
authorKent Hansen <kent.hansen@nokia.com>
Mon, 20 Aug 2012 10:40:36 +0000 (12:40 +0200)
committerQt by Nokia <qt-info@nokia.com>
Wed, 22 Aug 2012 07:11:30 +0000 (09:11 +0200)
commit9429a85a2160cf8b860b1292e4e0bc0855841755
treeb255ba0f8f5e895fcc3c3cd90fd31d7ca391a143
parent75c649276a9bbcb2e593d6c6f78d77417dd05891
Capture full backtrace when a native function calls back into JS

Commit df0ec196031d33850324dc5eeed2d71f61413885 assumed that JSC's
Interpreter::throwException function is called exactly once when an
exception occurs. That's wrong. If there is a native call inbetween
two JS calls on the stack, the call stack will be unwound to the
point of the native call frame, and throwException will return. After
the native (C) call frame has been unwound, throwException will be
called again to unwind remaining JS call frames, and so on.

This was causing QtScript to discard the backtrace belonging to the
inner-most JS frames; the backtrace would be regenerated from a
partially unwound state.

Fix this by ignoring subsequent calls to the uncaughtException()
callback once a backtrace has been captured; the backtrace is already
cleared before each evaluation is started.

Task-number: QTBUG-26889
Change-Id: I03e1d60fbac5e592cff1dd5ef70f397cf94454ae
Reviewed-by: Simon Hausmann <simon.hausmann@nokia.com>
src/script/api/qscriptengine.cpp
tests/auto/qscriptengine/tst_qscriptengine.cpp