• Simon Zünd's avatar
    Store JSMessageObject on rejected promises · b15c02d0
    Simon Zünd authored
    When V8 throws an uncaught exception, we store a JSMessageObject
    with a stack trace and source positions on the isolate itself.
    The JSMessageObject can be retrieved by a TryCatch scope
    and is used by the inspector to provide additional information to the DevTools
    frontend (besides the exception).
    
    Introducing top-level await for REPL mode causes all thrown exceptions
    to be turned into a rejected promise. The implicit catch block that does this
    conversion clears the JSMessageObject from the isolate as to not leak memory.
    
    This CL preserves the JSMessageObject when the debugger is active and stores
    the JSMessageObject on the rejected promise itself. The inspector is changed
    to retrieve the JSMessageObject in the existing catch handler and pass the
    information along to the frontend.
    
    Drive-by: This CL removes a inspector test that made assumptions when a promise
    is cleaned up by the GC. These assumptions no longer hold since we hold on to
    the promise longer.
    
    Bug: chromium:1021921
    Change-Id: Id0380e2cf3bd79aca05191bc4f3c616f6ced8db7
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1967375
    Commit-Queue: Simon Zünd <szuend@chromium.org>
    Reviewed-by: 's avatarLeszek Swirski <leszeks@chromium.org>
    Reviewed-by: 's avatarPeter Marshall <petermarshall@chromium.org>
    Reviewed-by: 's avatarBenedikt Meurer <bmeurer@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#65497}
    b15c02d0
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer Loading commit data...
inspector Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mkgrokdump Loading commit data...
mozilla Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...