Add method for getting global v8::Context

Exposed method for finding global context by traversing context chain to API.

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

[Søren Thygesen Gjesse, 2009-05-19 12:08:46]

This is not the way to go, as the contexts exposed through the API are always
global contexts. It is the GetEventContext on the Message object which is in
error, as it returns a non-global context.

I think we can remove the GetEventContext from the Message object and use
Context::GetCurrent() in the API instead.

[yurys, 2009-05-19 17:10:29]

On 2009/05/19 12:08:46, Søren Gjesse wrote:
> This is not the way to go, as the contexts exposed through the API are always
> global contexts. It is the GetEventContext on the Message object which is in
> error, as it returns a non-global context.
> 
> I think we can remove the GetEventContext from the Message object and use
> Context::GetCurrent() in the API instead.


As discussed offline we cannot use Context::GetCurrent() as it would return the
debugger context. I've changed implementation of GetEventContext so that it
looks up global context by traversing context chain that was active right before
entering the debugger.

[Søren Thygesen Gjesse, 2009-05-20 07:24:02]

LGTM!

Thanks for fixing this (and finding the simpler and right solution) and for
adding the tests.

http://codereview.chromium.org/112035/diff/2007/3014
File test/cctest/test-debug.cc (right):

http://codereview.chromium.org/112035/diff/2007/3014#newcode4175
Line 4175: // Sends conutinue command to the debugger.
conutinue -> continue

[yurys, 2009-05-20 07:28:38]

http://codereview.chromium.org/112035/diff/2007/3014
File test/cctest/test-debug.cc (right):

http://codereview.chromium.org/112035/diff/2007/3014#newcode4175
Line 4175: // Sends conutinue command to the debugger.
On 2009/05/20 07:24:02, Søren Gjesse wrote:
> conutinue -> continue

Done.