Debugger: require debugger to be active when dealing with breaks.
authoryangguo <yangguo@chromium.org>
Tue, 16 Jun 2015 07:11:11 +0000 (00:11 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 16 Jun 2015 07:11:21 +0000 (07:11 +0000)
commit75350f1ef0ebf189ef3583183eae694a25597fc0
tree7fca124457ecfe529484f0d8cafff64f5cdd8770
parent2c44f4f9deb4015555b7825c81861b06c33f346d
Debugger: require debugger to be active when dealing with breaks.

This invariant will save us some head ache.
The changes to test-debug/DebugStub is due to the fact that it abuses
the ability to set break points in code that has no debug break slots.
This is now no longer possible.

R=ulan@chromium.org
BUG=v8:4132
LOG=N

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

Cr-Commit-Position: refs/heads/master@{#29038}
22 files changed:
src/debug.cc
src/debug.h
src/runtime/runtime-debug.cc
test/cctest/cctest.h
test/cctest/test-api.cc
test/cctest/test-debug.cc
test/cctest/test-heap.cc
test/mjsunit/debug-liveedit-breakpoints.js
test/mjsunit/debug-liveedit-patch-positions.js
test/mjsunit/debug-liveedit-stack-padding.js
test/mjsunit/debug-script-breakpoints.js
test/mjsunit/debug-script.js
test/mjsunit/debug-stepin-positions.js
test/mjsunit/deserialize-script-id.js
test/mjsunit/es6/debug-evaluate-blockscopes.js
test/mjsunit/es6/generators-relocation.js
test/mjsunit/regress/regress-1586.js
test/mjsunit/regress/regress-2318.js
test/mjsunit/regress/regress-crbug-424142.js
test/mjsunit/regress/regress-crbug-474297.js
test/mjsunit/regress/regress-debug-code-recompilation.js
test/mjsunit/regress/regress-prepare-break-while-recompile.js