Initial implementation of printing the AST as a JSON string. This...

Initial implementation of printing the AST as a JSON string.  This
implementation is not yet complete (it prints only the node name for
some AST constructs) and does not serialize everything needed to
reconstitute the AST.

It is motivated by a desire to prototype source-to-source
transformations in JavaScript itself (or anything else that can grok
JSON), but it should have other uses too.

Feedback is welcome.

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

[mnaganov (inactive), 2009-06-19 10:20:11]

LGTM

I'm curious, what will you do with a transformed AST? Just look at it, or are
you planning to implement AST builder from JSON as well?

http://codereview.chromium.org/131101/diff/1/6
File src/codegen.cc (right):

http://codereview.chromium.org/131101/diff/1/6#newcode174
Line 174: PrintF(builder.BuildProgram(flit));
I'd suggest rewriting this as PrintF("%s", builder.BuildProgram(flit))

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

http://codereview.chromium.org/131101/diff/1/2#newcode1183
Line 1183: Print(text);
Hint: You can write Print("%*s", indent_, text) (use field width to indent with
spaces).

[mnaganov (inactive), 2009-06-19 11:53:39]

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

http://codereview.chromium.org/131101/diff/1/2#newcode1183
Line 1183: Print(text);
On 2009/06/19 10:20:11, Michail Naganov wrote:
> Hint: You can write Print("%*s", indent_, text) (use field width to indent
with
> spaces).

Sorry, I misguided you, the right code is: Print("%*s%s", indent_, "", text)

[Søren Thygesen Gjesse, 2009-07-01 16:46:11]

LGTM

Should we add something to the debugger protocol to retrieve the AST?

[Kevin Millikin (Chromium), 2009-10-12 14:14:45]

I've addressed the comments and resurrected this to land it on bleeding_edge. 
It's still incomplete.  It probably needs another quick look.

[fschneider, 2009-10-12 14:51:47]

LGTM!