[Isolates] Fix debugger and termination APIs callable from other threads.

[Isolates] Fix debugger and termination APIs callable from other threads.

A couple of debugger methods and one of the execution termination methods can be called from any thread. If they do not supply an isolate, use the default isolate. This ensures backwards compatibility and gives the possibility of specifying an isolate.

R=vitalyr@chromium.org

Committed: http://code.google.com/p/v8/source/detail?r=7208

[Vitaly Repeshko, 2011-03-16 18:08:21]

LGTM

http://codereview.chromium.org/6672044/diff/1/include/v8-debug.h
File include/v8-debug.h (right):

http://codereview.chromium.org/6672044/diff/1/include/v8-debug.h#newcode257
include/v8-debug.h:257: static void DebugBreak(Isolate* isolate = NULL);
Please document that these functions use the default isolate if none given.

http://codereview.chromium.org/6672044/diff/1/include/v8.h
File include/v8.h (right):

http://codereview.chromium.org/6672044/diff/1/include/v8.h#newcode2951
include/v8.h:2951: static void TerminateExecution(Isolate* isolate = NULL);
Same here.

[Mads Ager (chromium), 2011-03-16 18:52:47]

http://codereview.chromium.org/6672044/diff/1/include/v8-debug.h
File include/v8-debug.h (right):

http://codereview.chromium.org/6672044/diff/1/include/v8-debug.h#newcode257
include/v8-debug.h:257: static void DebugBreak(Isolate* isolate = NULL);
On 2011/03/16 18:08:21, Vitaly Repeshko wrote:
> Please document that these functions use the default isolate if none given.

Done.

http://codereview.chromium.org/6672044/diff/1/include/v8.h
File include/v8.h (right):

http://codereview.chromium.org/6672044/diff/1/include/v8.h#newcode2951
include/v8.h:2951: static void TerminateExecution(Isolate* isolate = NULL);
On 2011/03/16 18:08:21, Vitaly Repeshko wrote:
> Same here.

Done.