includes: Make ast.h not need isolate.h any more.

includes: Make ast.h not need isolate.h any more.

Also remove other unnecessary includes from ast.h and dependencies.

BUG=v8:5294
Committed: https://crrev.com/9c888dffda4bff420ec3ef21e623aa4e5054d1c3
Cr-Commit-Position: refs/heads/master@{#39155}

[marja, 2016-09-02 12:32:00]

Description was changed from

==========
includes: Make ast.h not need isolate.h any more.

BUG=v8:5294
==========

to

==========
includes: Make ast.h not need isolate.h any more.

Also remove other unnecessary includes from ast.h and dependencies.


BUG=v8:5294
==========

[marja, 2016-09-02 12:32:00]

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

[marja, 2016-09-02 12:32:50]

adamk, ptal

(Sadly, this doesn't solve the problem in the bug (the problem is the other way
around, something pulling in ast.h et al into everything), I just happened to
bump into this along the way...)

[marja, 2016-09-02 12:33:04]

The CQ bit was checked by marja@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-02 12:33:11]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-02 13:10:25]

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

[commit-bot: I haz the power, 2016-09-02 13:10:26]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-09-02 17:13:20]

lgtm

[marja, 2016-09-05 07:42:25]

The CQ bit was checked by marja@chromium.org

[commit-bot: I haz the power, 2016-09-05 07:42:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-05 07:43:43]

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

[commit-bot: I haz the power, 2016-09-05 07:43:43]

Try jobs failed on following builders:
  v8_android_arm_compile_rel on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_android_arm_compile_rel/...)
  v8_linux_dbg_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_dbg_ng/builds/12104)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/8179)

[marja, 2016-09-05 09:08:24]

Patchset #3 (id:40001) has been deleted

[marja, 2016-09-05 09:08:28]

The CQ bit was checked by marja@chromium.org

[marja, 2016-09-05 09:08:28]

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

[commit-bot: I haz the power, 2016-09-05 09:08:30]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-05 09:12:19]

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

[commit-bot: I haz the power, 2016-09-05 09:12:20]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/23260)

[Michael Starzinger, 2016-09-05 09:19:16]

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

[Michael Starzinger, 2016-09-05 09:19:17]

LGTM.

[marja, 2016-09-05 09:21:32]

The CQ bit was checked by marja@chromium.org

[commit-bot: I haz the power, 2016-09-05 09:21:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-05 09:25:29]

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

[commit-bot: I haz the power, 2016-09-05 09:25:30]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/23263)

[Benedikt Meurer, 2016-09-05 09:49:30]

bmeurer@chromium.org changed reviewers:
+ bmeurer@chromium.org

[Benedikt Meurer, 2016-09-05 09:49:31]

LGTM (rubberstamp)

[marja, 2016-09-05 09:50:36]

The CQ bit was checked by marja@chromium.org

[commit-bot: I haz the power, 2016-09-05 09:50:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-05 09:55:16]

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

[commit-bot: I haz the power, 2016-09-05 09:55:17]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/23267)

[titzer, 2016-09-05 10:39:30]

On 2016/09/05 09:55:17, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   v8_presubmit on master.tryserver.v8 (JOB_FAILED,
> http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/23267)

lgtm

[marja, 2016-09-05 10:39:50]

The CQ bit was checked by marja@chromium.org

[commit-bot: I haz the power, 2016-09-05 10:39:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-05 10:53:11]

Description was changed from

==========
includes: Make ast.h not need isolate.h any more.

Also remove other unnecessary includes from ast.h and dependencies.


BUG=v8:5294
==========

to

==========
includes: Make ast.h not need isolate.h any more.

Also remove other unnecessary includes from ast.h and dependencies.


BUG=v8:5294
==========

[commit-bot: I haz the power, 2016-09-05 10:53:12]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2016-09-05 10:53:35]

Description was changed from

==========
includes: Make ast.h not need isolate.h any more.

Also remove other unnecessary includes from ast.h and dependencies.


BUG=v8:5294
==========

to

==========
includes: Make ast.h not need isolate.h any more.

Also remove other unnecessary includes from ast.h and dependencies.

BUG=v8:5294

Committed: https://crrev.com/9c888dffda4bff420ec3ef21e623aa4e5054d1c3
Cr-Commit-Position: refs/heads/master@{#39155}
==========

[commit-bot: I haz the power, 2016-09-05 10:53:36]

Patchset 3 (id:??) landed as
https://crrev.com/9c888dffda4bff420ec3ef21e623aa4e5054d1c3
Cr-Commit-Position: refs/heads/master@{#39155}

[Michael Achenbach, 2016-09-05 11:48:35]

A revert of this CL (patchset #3 id:60001) has been created in
https://codereview.chromium.org/2305373002/ by machenbach@chromium.org.

The reason for reverting is: Breaks gc stress:
https://build.chromium.org/p/client.v8/builders/V8%20Linux%20-%20gc%20stress/....