Added context information to scripts

Added context information to scripts.

Added a simple initial version of a context mirror object which just holds the user data from the context, and does not preserve handle identity.

A script object now holds a reference to the custom data from the context it was compiled in. This data is included in the debugger protocol for scripts.

Changed the serialization for the scripts command to use the mirror objects instaed of custom serialization. This included passing options from the debugger request to the serializer for including the full source of scripts in the response.

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

[Søren Thygesen Gjesse, 2009-05-05 14:59:10]

I was trying to just reference the context from the script, but that leaked
global objects in some of the tests.

There are still tests missing

[yurys, 2009-05-06 07:22:34]

LGTM, though having a reference from the script object to the context and
requesting data from the context would be more flexible. We would be able to set
the context data only for the contexts we were going to debug. With this
implementation we would need to setup context data as early as possible to
guarantee the data is already set when the script is being compiled.

http://codereview.chromium.org/111001/diff/1/6
File src/mirror-delay.js (right):

http://codereview.chromium.org/111001/diff/1/6#newcode1845
Line 1845: content.push(MakeJSONPair_('sourceLength',
it seems that you can switch to JSON.stringify for serialization and get rid of
all the custom json serializers(MakeJSONPair_, *ToJSON_, etc)

[Søren Thygesen Gjesse, 2009-05-06 07:48:54]

Having a reference from the script to the context should be possible, however
just adding the reference caused leakage of global objects. I made this approach
to get something for you to work with while resolving that issue.

http://codereview.chromium.org/111001/diff/1/6
File src/mirror-delay.js (right):

http://codereview.chromium.org/111001/diff/1/6#newcode1845
Line 1845: content.push(MakeJSONPair_('sourceLength',
On 2009/05/06 07:22:34, Yury Semikhatsky wrote:
> it seems that you can switch to JSON.stringify for serialization and get rid
of
> all the custom json serializers(MakeJSONPair_, *ToJSON_, etc)

Sure, that would clean up the code quite a bit.

[Christian Plesner Hansen, 2009-05-06 08:29:21]

Lgtm!