Adding GC hooks to support LiveObjectList functionality.

Adding GC hooks to support LiveObjectList functionality.

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

[Søren Thygesen Gjesse, 2011-01-19 08:41:33]

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

http://codereview.chromium.org/6287004/diff/1/src/heap.cc#newcode38
src/heap.cc:38: #include "liveobjectlist-inl.h"
This suggests that the actual lol implementation lands before this change.

How about a separate change with the empty implementation from
http://codereview.chromium.org/6357005/ which can land first. Then we can land
the this change and the debugger support change and then finally the actual lol
implementation?

[marklam, 2011-01-19 10:47:37]

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

http://codereview.chromium.org/6287004/diff/1/src/heap.cc#newcode38
src/heap.cc:38: #include "liveobjectlist-inl.h"
On 2011/01/19 08:41:33, Søren Gjesse wrote:
> This suggests that the actual lol implementation lands before this change.
> 
> How about a separate change with the empty implementation from
> http://codereview.chromium.org/6357005/ which can land first. Then we can land
> the this change and the debugger support change and then finally the actual
lol
> implementation?

Done.  It's at http://codereview.chromium.org/6335009/.

[Søren Thygesen Gjesse, 2011-01-25 12:35:54]

LGTM

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