Code cleanup & simplification....

Code cleanup & simplification.
Moves the calls to tracer and UpdateLiveObjectCount inside the call to
(inlined) SetMark.  Removes global object counter.

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

[William Hesse, 2008-10-30 11:18:34]

As part of inlining the tracing and debug object-counting into
SetMark during marking GC, I dropped the global objects V8 counter.  It wasn't
being updated at all SetMark sites, and people here said it wasn't being used. 
Do either of you really want to keep it?

The inlined code does inline, I checked the disassembly.  Please review.

[Patrick Johnson, 2008-10-30 16:04:25]

LGTM

I only reviewed the change to remove the global objects counter.  We're not
using it either as far as I know.

[Feng Qian, 2008-10-30 16:36:51]

Global object counter is pretty useful for debugging memory leaks.
Does it cause trouble for inlining?

On 2008/10/30 16:04:25, Patrick Johnson wrote:
> LGTM
> 
> I only reviewed the change to remove the global objects counter.  We're not
> using it either as far as I know.

[Kevin Millikin (Chromium), 2008-11-04 09:59:32]

LGTM.