Remove unused Zone argument from InitializeAstVisitor

Remove unused Zone argument from InitializeAstVisitor

This adds a bit of boilerplate to some AstVisitors (they now have to
declare their own zone_ member and zone() accessor), but makes it clearer
what DEFINE_AST_VISITOR_SUBCLASS_MEMBERS is for: stack limit checking.

Committed: https://crrev.com/4937cc9457aac003b71b59cfc7683b0c18836c6a
Cr-Commit-Position: refs/heads/master@{#31287}

[adamk, 2015-10-14 15:54:36]

adamk@chromium.org changed reviewers:
+ mstarzinger@chromium.org

[adamk, 2015-10-14 15:54:37]

Quick followup cleanup to the Isolate one (if you'd rather just keep the macro
I'm ok with it too).

[Michael Starzinger, 2015-10-15 07:58:51]

LGTM.

https://codereview.chromium.org/1394303008/diff/1/src/ast-expression-visitor.cc
File src/ast-expression-visitor.cc (right):

https://codereview.chromium.org/1394303008/diff/1/src/ast-expression-visitor....
src/ast-expression-visitor.cc:35:
AstExpressionVisitor::AstExpressionVisitor(Isolate* isolate, Zone* zone,
nit: Looks like the "zone" argument is no longer needed here either.

[adamk, 2015-10-15 08:03:40]

https://codereview.chromium.org/1394303008/diff/1/src/ast-expression-visitor.cc
File src/ast-expression-visitor.cc (right):

https://codereview.chromium.org/1394303008/diff/1/src/ast-expression-visitor....
src/ast-expression-visitor.cc:35:
AstExpressionVisitor::AstExpressionVisitor(Isolate* isolate, Zone* zone,
On 2015/10/15 07:58:51, Michael Starzinger wrote:
> nit: Looks like the "zone" argument is no longer needed here either.

Done.

[adamk, 2015-10-15 08:03:59]

The CQ bit was checked by adamk@chromium.org

[adamk, 2015-10-15 08:03:59]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1394303008/#ps20001
(title: "Fix AstExpressionVisitor")

[commit-bot: I haz the power, 2015-10-15 08:04:08]

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

[adamk, 2015-10-15 08:04:58]

The CQ bit was unchecked by adamk@chromium.org

[adamk, 2015-10-15 08:05:19]

Hmm, cq bit maybe was a bit hasty, looks like I somehow caused crashes.

[adamk, 2015-10-15 08:23:03]

The CQ bit was checked by adamk@chromium.org

[adamk, 2015-10-15 08:23:04]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1394303008/#ps40001
(title: "Fix build and AstTyper constructor")

[commit-bot: I haz the power, 2015-10-15 08:23:23]

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

[commit-bot: I haz the power, 2015-10-15 08:26:31]

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

[commit-bot: I haz the power, 2015-10-15 08:26:32]

Try jobs failed on following builders:
  v8_linux_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel/builds/10554)
  v8_mac_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel/builds/10691)

[adamk, 2015-10-15 09:50:28]

The CQ bit was checked by adamk@chromium.org

[adamk, 2015-10-15 09:50:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1394303008/#ps60001
(title: "Fix cctests")

[commit-bot: I haz the power, 2015-10-15 09:50:42]

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

[commit-bot: I haz the power, 2015-10-15 10:34:14]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-15 10:34:38]

Patchset 4 (id:??) landed as
https://crrev.com/4937cc9457aac003b71b59cfc7683b0c18836c6a
Cr-Commit-Position: refs/heads/master@{#31287}