Enable lazy compilation for non-trivial outer contexts.

Enable lazy compilation for non-trivial outer contexts.

This changes the compiler to be more aggressive about lazy compilation
of closures with non-trivial outer context. Compilation can only be
triggered with a valid outer context now. One exception is the debugger,
which can request compilation of arbitrary shared code, but it ensures
to trigger compilation only at points where no context is needed.

R=ulan@chromium.org
TEST=mjsunit/debug-script-breakpoints-nested

Committed: https://code.google.com/p/v8/source/detail?r=11782

[ulan, 2012-06-12 15:07:23]

LGTM

https://chromiumcodereview.appspot.com/10538102/diff/1/src/debug.cc
File src/debug.cc (right):

https://chromiumcodereview.appspot.com/10538102/diff/1/src/debug.cc#newcode2122
src/debug.cc:2122: has_break_points_ = true;
As discussed offline, we can turn this to ASSERT(has_break_points);

https://chromiumcodereview.appspot.com/10538102/diff/1/src/scopes.cc
File src/scopes.cc (right):

https://chromiumcodereview.appspot.com/10538102/diff/1/src/scopes.cc#newcode675
src/scopes.cc:675: bool Scope::AllowsLazyRecompilation() const {
Now that it is the same as AllowsLazyCompilation, we can probably get rid of it.

https://chromiumcodereview.appspot.com/10538102/diff/1/test/mjsunit/debug-scr...
File test/mjsunit/debug-script-breakpoints-nested.js (right):

https://chromiumcodereview.appspot.com/10538102/diff/1/test/mjsunit/debug-scr...
test/mjsunit/debug-script-breakpoints-nested.js:49: "    function b() {\n" +
Consider adding more local variables.

[Michael Starzinger, 2012-06-12 15:54:05]

https://chromiumcodereview.appspot.com/10538102/diff/1/src/debug.cc
File src/debug.cc (right):

https://chromiumcodereview.appspot.com/10538102/diff/1/src/debug.cc#newcode2122
src/debug.cc:2122: has_break_points_ = true;
On 2012/06/12 15:07:23, ulan wrote:
> As discussed offline, we can turn this to ASSERT(has_break_points);

Done.

https://chromiumcodereview.appspot.com/10538102/diff/1/src/scopes.cc
File src/scopes.cc (right):

https://chromiumcodereview.appspot.com/10538102/diff/1/src/scopes.cc#newcode675
src/scopes.cc:675: bool Scope::AllowsLazyRecompilation() const {
On 2012/06/12 15:07:23, ulan wrote:
> Now that it is the same as AllowsLazyCompilation, we can probably get rid of
it.

Done.

https://chromiumcodereview.appspot.com/10538102/diff/1/test/mjsunit/debug-scr...
File test/mjsunit/debug-script-breakpoints-nested.js (right):

https://chromiumcodereview.appspot.com/10538102/diff/1/test/mjsunit/debug-scr...
test/mjsunit/debug-script-breakpoints-nested.js:49: "    function b() {\n" +
On 2012/06/12 15:07:23, ulan wrote:
> Consider adding more local variables.

Done.