Remove check for heap allocated locals from code flushing....

Remove check for heap allocated locals from code flushing.

Since bleeding edge revision 5056 we no longer need to check that
there are no heap allocated locals in the scope info, since these will
be preserved after the scope info was moved to the SharedFunctionInfo
class.



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

[Kasper Lund, 2010-07-14 07:00:30]

LGTM.

http://codereview.chromium.org/2931016/diff/1/2
File src/heap.cc (right):

http://codereview.chromium.org/2931016/diff/1/2#newcode2484
src/heap.cc:2484: // Compute the lazy compilable version of the code, clear the
scope info.
Update comment (scope info isn't cleared). Maybe move the comment above the
HandleScope declaration. That looks better to me.

[Rico, 2010-07-14 07:10:44]

http://codereview.chromium.org/2931016/diff/1/2
File src/heap.cc (right):

http://codereview.chromium.org/2931016/diff/1/2#newcode2484
src/heap.cc:2484: // Compute the lazy compilable version of the code, clear the
scope info.
On 2010/07/14 07:00:30, Kasper Lund wrote:
> Update comment (scope info isn't cleared). Maybe move the comment above the
> HandleScope declaration. That looks better to me.

Done.