Add the ability to set embedder data on created contexts from the API

Add the ability to set embedder data on created contexts from the API.

Expose the active context where a break event occoured through the debug message handler.

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

[Søren Thygesen Gjesse, 2009-05-05 09:09:47]

This is the first of a series of changes to provide more information on the
context to the debugger.

[Christian Plesner Hansen, 2009-05-05 09:19:06]

Lgtm.

A general comment: I've been using the name 'Data' pretty consistently, the same
way you do here.  However, now I'm thinking that maybe it would be better to
call is UserData to make it absolutely clear what it is.  Just an idea.

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

http://codereview.chromium.org/109013/diff/1/3#newcode2268
Line 2268: Handle<Value> GetData();
If it returns a local handle consider using the type Local<Value> instead.

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

http://codereview.chromium.org/109013/diff/1/4#newcode451
Line 451: {
Why the inner scope?

[Søren Thygesen Gjesse, 2009-05-05 09:32:01]

As discussed offline changing Data to UserData is th right thing to do. However
it should be changed consistently in all places at some point.

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

http://codereview.chromium.org/109013/diff/1/3#newcode2268
Line 2268: Handle<Value> GetData();
On 2009/05/05 09:19:07, Christian Plesner Hansen wrote:
> If it returns a local handle consider using the type Local<Value> instead.

Done.

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

http://codereview.chromium.org/109013/diff/1/4#newcode451
Line 451: {
On 2009/05/05 09:19:07, Christian Plesner Hansen wrote:
> Why the inner scope?

To avoid adding the "env" handle to the outer scope.