Add ENABLE_DEBUGGER_SUPPORT macro....

Add ENABLE_DEBUGGER_SUPPORT macro.

ENABLE_DEBUGGER_SUPPORT is enabled by default unless it is on Android platform.
On Android platform, it can also enabled by passing -DENABLE_DEBUGGER_SUPPORT flag to the compiler.

This should not affect any existing build (I hope, cross my fingers) except the build in real Android environment (in other word, it only affects me now).

There are lot of room for code refactoring in stead of using #ifdef all over the place. I will leave this to v8 folks.


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

[Søren Thygesen Gjesse, 2009-04-17 23:46:17]

LGTM

It is nasty with all these #ifdef's, but they are somewhat grouped together. I
will refactor to get rid of them going forward.

You should remember to exclude debug-delay.js and mirror-delay.js when building
without debugger enabled.

http://codereview.chromium.org/77035/diff/1/32
File src/serialize.cc (right):

http://codereview.chromium.org/77035/diff/1/32#newcode683
Line 683: 5,
Why not re-number these?

http://codereview.chromium.org/77035/diff/1/45
File src/stub-cache.h (right):

http://codereview.chromium.org/77035/diff/1/45#newcode297
Line 297: Object* CompileLazyCompile(Code::Flags flags);
Please move this before the #ifdef ENABLE_DEBUGGER_SUPPORT.

http://codereview.chromium.org/77035/diff/1/22
File src/v8threads.cc (right):

http://codereview.chromium.org/77035/diff/1/22#newcode178
Line 178: Debug::ArchiveSpacePerThread() +
Please move this to the bottom of the "list".

http://codereview.chromium.org/77035/diff/1/22#newcode267
Line 267: to = Debug::ArchiveDebug(to);
Please move this to the bottom of the "list".