Provide access to the debuggee context from debug event listener

Provide debug event listener with access to the debuggee context. Also introduce new event listener setter that allows to set a callback that accepts single parameter encapsulating all debug event details so that additional information can later be passed to the listener without breaking compatibility with existing clients.


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

[Søren Thygesen Gjesse, 2010-05-20 14:42:23]

LGTM

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

http://codereview.chromium.org/2108024/diff/1/2#newcode198
include/v8-debug.h:198: * A EventCallback2 does not take posession of the event
data,
posession -> possession

http://codereview.chromium.org/2108024/diff/1/4
File src/debug.cc (right):

http://codereview.chromium.org/2108024/diff/1/4#newcode2128
src/debug.cc:2128: printf("xxx %d\n", event_listener_data_->IsUndefined());
Please remove printf.

[yurys, 2010-05-20 17:06:34]

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

http://codereview.chromium.org/2108024/diff/1/2#newcode198
include/v8-debug.h:198: * A EventCallback2 does not take posession of the event
data,
On 2010/05/20 14:42:23, Søren Gjesse wrote:
> posession -> possession

Done.

http://codereview.chromium.org/2108024/diff/1/4
File src/debug.cc (right):

http://codereview.chromium.org/2108024/diff/1/4#newcode2128
src/debug.cc:2128: printf("xxx %d\n", event_listener_data_->IsUndefined());
On 2010/05/20 14:42:23, Søren Gjesse wrote:
> Please remove printf.

Oops, done.