Postpone interrupts while dipatching debugger events to listeners
authoryurys <yurys@chromium.org>
Mon, 31 Aug 2015 22:32:46 +0000 (15:32 -0700)
committerCommit bot <commit-bot@chromium.org>
Mon, 31 Aug 2015 22:32:56 +0000 (22:32 +0000)
commit70dc24c22ff61b4b4d5a9f43597bdbb067b35304
tree6dd42610e5e26f071cd1f490506610a2587b1006
parent206f12abee3f1e7eda8fc6521d48f3c319460ee1
Postpone interrupts while dipatching debugger events to listeners

The interrupts are already postponed in message handlers [1]. This CL aligns debug event listener (the mechanism that is actually used in Chrome DevTools) implementation with that. Handling interrupts on events like v8::AfterCompile leads to crashes like the one in the lined bug. This happens because in the interrupt handler we may change debugger state.

[1] https://codereview.chromium.org/309533009/diff/40001/src/debug.cc

BUG=chromium:520702
LOG=Y

Review URL: https://codereview.chromium.org/1321263002

Cr-Commit-Position: refs/heads/master@{#30488}
src/debug/debug.cc
test/cctest/test-debug.cc