Improve debugger property lookup

Improve debugger property lookup.

before performing debugger property lookup make sure the current context is set to the context active before the debugger was entered.

Make the use of the LookupResult GC safe in debugger property lookup.

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

[Søren Thygesen Gjesse, 2009-05-28 08:11:00]

Could you please check whether this change gets rid of the crashes you have been
experiencing yesterday?

[yurys, 2009-05-28 08:48:21]

LGTM, we will check if this fixes the craches we ovserved yesterday in the
remote Eclipse debugger when Alexander is in office.

http://codereview.chromium.org/115855/diff/1001/2001
File src/runtime.cc (left):

http://codereview.chromium.org/115855/diff/1001/2001#oldcode5522
Line 5522: if (Debug::debugger_entry()) {
remove the 'if' since both branches are equal

[Søren Thygesen Gjesse, 2009-05-28 11:16:55]

http://codereview.chromium.org/115855/diff/1001/2001
File src/runtime.cc (left):

http://codereview.chromium.org/115855/diff/1001/2001#oldcode5522
Line 5522: if (Debug::debugger_entry()) {
On 2009/05/28 08:48:21, Yury Semikhatsky wrote:
> remove the 'if' since both branches are equal

Done.

[apavlov, 2009-05-28 11:45:43]

Thanks for the patch. It does fix the crash for me!