Fixed the debugger compile events....

Fixed the debugger compile events.

The debugger compile events (BeforeCompile and AfterCompile) have not been used by any V8 debuggers lately and where actually not working any more. Added the correct information to the compile event.

Added a simple 'trace compile' command to the developer shell for testing.

Added a test for compile evnets.

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

[Mads Ager (chromium), 2009-02-05 09:56:40]

LGTM

http://codereview.chromium.org/21076/diff/1/5
File src/d8.js (right):

http://codereview.chromium.org/21076/diff/1/5#newcode106
Line 106: if (event.eventType() == Debug.DebugEvent.Break) {
How about a switch on the event type instead?

http://codereview.chromium.org/21076/diff/1/2
File test/mjsunit/debug-compile-event.js (right):

http://codereview.chromium.org/21076/diff/1/2#newcode1
Line 1: // Copyright 2008 the V8 project authors. All rights reserved.
2009

http://codereview.chromium.org/21076/diff/1/2#newcode35
Line 35: 
Why put spaces here and not before the function?

[Søren Thygesen Gjesse, 2009-02-05 10:09:15]

http://codereview.chromium.org/21076/diff/1/5
File src/d8.js (right):

http://codereview.chromium.org/21076/diff/1/5#newcode106
Line 106: if (event.eventType() == Debug.DebugEvent.Break) {
On 2009/02/05 09:56:40, Mads Ager wrote:
> How about a switch on the event type instead?

Done.

http://codereview.chromium.org/21076/diff/1/2
File test/mjsunit/debug-compile-event.js (right):

http://codereview.chromium.org/21076/diff/1/2#newcode1
Line 1: // Copyright 2008 the V8 project authors. All rights reserved.
On 2009/02/05 09:56:40, Mads Ager wrote:
> 2009

Done.

http://codereview.chromium.org/21076/diff/1/2#newcode35
Line 35: 
On 2009/02/05 09:56:40, Mads Ager wrote:
> Why put spaces here and not before the function?

Tried to indicate that these variables where set by the function. Did not work.
Changed formatting and added comments.