Adding AstNode::PrettyPrint and AstNode::PrintAst

Adding AstNode::PrettyPrint and AstNode::PrintAst

BUG=

Committed: https://crrev.com/b0119907ac8ffb366bb3a9f684a7a38720347523
Cr-Commit-Position: refs/heads/master@{#33323}

[Camillo Bruni, 2016-01-13 14:27:12]

cbruni@chromium.org changed reviewers:
+ littledan@chromium.org

[Camillo Bruni, 2016-01-13 14:27:12]

PTAL, after being stuck on where inition would fail I added these two helper
methods.

[Dan Ehrenberg, 2016-01-13 14:44:57]

Could you give more details on what problem this is trying to solve?

[Camillo Bruni, 2016-01-14 09:16:11]

On 2016/01/13 at 14:44:57, littledan wrote:
> Could you give more details on what problem this is trying to solve?

I think I come up with this idea every month :).
Most probably this patch is not very useful for people dealing with the ast
regularly.
Yesterday I was running into an issue with the interpreter failing on a certain
ast-node.
It took me 5mins to figure out how to get back to the source code to figure out
where
we actually failed (note, there is no direct reference to Print in
ast.cc/ast.h). I like
the idea of having a Print() method on every internal v8 structure as helpers.

[Dan Ehrenberg, 2016-01-14 18:18:39]

lgtm

[Camillo Bruni, 2016-01-15 10:00:48]

The CQ bit was checked by cbruni@chromium.org

[commit-bot: I haz the power, 2016-01-15 10:00:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1587543003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1587543003/1

[commit-bot: I haz the power, 2016-01-15 10:02:39]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-01-15 10:02:39]

Try jobs failed on following builders:
  v8_android_arm_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux_mips64el_compile_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_mips64el_compile_r...)
  v8_linux_nodcheck_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel/build...)

[Camillo Bruni, 2016-01-15 10:52:33]

The CQ bit was checked by cbruni@chromium.org

[Camillo Bruni, 2016-01-15 10:52:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
littledan@chromium.org
Link to the patchset: https://codereview.chromium.org/1587543003/#ps20001
(title: "don't generate methods in release")

[commit-bot: I haz the power, 2016-01-15 10:52:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1587543003/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1587543003/20001

[commit-bot: I haz the power, 2016-01-15 11:32:30]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-01-15 11:33:42]

Description was changed from

==========
Adding AstNode::PrettyPrint and AstNode::PrintAst

BUG=
==========

to

==========
Adding AstNode::PrettyPrint and AstNode::PrintAst

BUG=

Committed: https://crrev.com/b0119907ac8ffb366bb3a9f684a7a38720347523
Cr-Commit-Position: refs/heads/master@{#33323}
==========

[commit-bot: I haz the power, 2016-01-15 11:33:43]

Patchset 2 (id:??) landed as
https://crrev.com/b0119907ac8ffb366bb3a9f684a7a38720347523
Cr-Commit-Position: refs/heads/master@{#33323}