Reorder SharedFunctionInfo setup in lazy compile, quick fix for crbug.com/490...

Reorder SharedFunctionInfo setup in lazy compile, quick fix for crbug.com/49099

This fixes issue 49099 (http://crbug.com/49099) that makes chromium
crash if a gc is triggered by setting the scopeinfo in CompileLazy. If
this gc triggers code flushing the compiled function could be flushed,
causing the ASSERT(shared->is_compiled()) to fail. By reordering the
two methods we make sure that no allocation happens before the end of
the method.


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

[Rico, 2010-07-19 06:24:56]

I will investigate further the test crashes that I found by manually introducing
a GC in the lazy compile function.

[Mads Ager (chromium), 2010-07-19 06:28:13]

LGTM