[parser] Deprecate ParseInfo constructor taking closure.

[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206
Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Committed: https://crrev.com/ab5379074d152c5841148277bfdd63739ecb9a4a
Cr-Original-Commit-Position: refs/heads/master@{#40083}
Cr-Commit-Position: refs/heads/master@{#40353}

[Michael Starzinger, 2016-10-06 11:47:19]

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

[commit-bot: I haz the power, 2016-10-06 11:47:24]

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

[Michael Starzinger, 2016-10-06 11:52:10]

Description was changed from

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206
==========

to

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206
==========

[Michael Starzinger, 2016-10-06 11:52:10]

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

[marja, 2016-10-06 11:55:48]

lgtm

[Michael Starzinger, 2016-10-06 12:03:52]

mstarzinger@chromium.org changed reviewers:
+ jarin@chromium.org, yangguo@chromium.org

[Michael Starzinger, 2016-10-06 12:03:52]

Jochen: PTAL at all the things.
Marja: PTAL at "parsing".
Yang: PTAL at "debug".
Jaro: PTAL at "crankshaft".

[commit-bot: I haz the power, 2016-10-06 12:20:12]

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

[commit-bot: I haz the power, 2016-10-06 12:20:12]

Dry run: This issue passed the CQ dry run.

[Benedikt Meurer, 2016-10-07 10:23:40]

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

[Benedikt Meurer, 2016-10-07 10:23:41]

LGTM on Crankshaft.

[Michael Starzinger, 2016-10-07 10:26:05]

mstarzinger@chromium.org changed reviewers:
- jarin@chromium.org

[jochen (gone - plz use gerrit), 2016-10-07 10:31:12]

all the things lgtm

[Michael Starzinger, 2016-10-07 10:34:52]

The CQ bit was checked by mstarzinger@chromium.org

[commit-bot: I haz the power, 2016-10-07 10:35:01]

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-10-07 10:39:13]

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

[commit-bot: I haz the power, 2016-10-07 10:39:14]

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/25918)

[Michael Starzinger, 2016-10-07 10:56:19]

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

[commit-bot: I haz the power, 2016-10-07 10:56:20]

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-10-07 11:22:50]

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

[commit-bot: I haz the power, 2016-10-07 11:22:50]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-07 11:55:25]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-10-07 11:55:25]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, jochen@chromium.org, marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2396963003/#ps20001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-10-07 11:55:35]

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-10-07 12:26:38]

Description was changed from

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206
==========

to

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206
==========

[commit-bot: I haz the power, 2016-10-07 12:26:39]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-07 12:27:03]

Description was changed from

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206
==========

to

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Cr-Commit-Position: refs/heads/master@{#40083}
==========

[commit-bot: I haz the power, 2016-10-07 12:27:04]

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

[Michael Hablich, 2016-10-07 21:13:03]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2406623002/ by hablich@chromium.org.

The reason for reverting is: Needed to revert
https://codereview.chromium.org/2400343002/.

[Michael Starzinger, 2016-10-17 09:54:22]

Description was changed from

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Cr-Commit-Position: refs/heads/master@{#40083}
==========

to

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Cr-Commit-Position: refs/heads/master@{#40083}
==========

[Michael Starzinger, 2016-10-17 09:56:00]

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

[commit-bot: I haz the power, 2016-10-17 09:56:13]

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-10-17 10:22:24]

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

[commit-bot: I haz the power, 2016-10-17 10:22:24]

Dry run: This issue passed the CQ dry run.

[Michael Starzinger, 2016-10-17 10:24:40]

The CQ bit was checked by mstarzinger@chromium.org

[Michael Starzinger, 2016-10-17 10:24:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
bmeurer@chromium.org, jochen@chromium.org, marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2396963003/#ps40001
(title: "Rebased.")

[commit-bot: I haz the power, 2016-10-17 10:24:45]

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-10-17 10:27:05]

Description was changed from

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Cr-Commit-Position: refs/heads/master@{#40083}
==========

to

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Cr-Commit-Position: refs/heads/master@{#40083}
==========

[commit-bot: I haz the power, 2016-10-17 10:27:06]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-17 10:27:32]

Description was changed from

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Cr-Commit-Position: refs/heads/master@{#40083}
==========

to

==========
[parser] Deprecate ParseInfo constructor taking closure.

This removes the {ParseInfo} constructor consuming a closure, replacing
all uses to pass only the shared function info. The goal is to make the
fact that parsing is independent of a concrete closure explicit.

R=jochen@chromium.org
BUG=v8:2206

Committed: https://crrev.com/3de42b3f224217ec88e4c609d3cf23fe06806dca
Committed: https://crrev.com/ab5379074d152c5841148277bfdd63739ecb9a4a
Cr-Original-Commit-Position: refs/heads/master@{#40083}
Cr-Commit-Position: refs/heads/master@{#40353}
==========

[commit-bot: I haz the power, 2016-10-17 10:27:33]

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