Re-scope inner scopes in arrow parameter initializers

Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
LOG=N

Committed: https://crrev.com/451fa77235e39a494b1abc3bb21a696373397e7e
Cr-Commit-Position: refs/heads/master@{#35542}

[rossberg, 2016-04-13 18:28:51]

lgtm

[nickie, 2016-04-15 14:08:02]

The CQ bit was checked by nikolaos@chromium.org

[commit-bot: I haz the power, 2016-04-15 14:08:12]

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

[nickie, 2016-04-15 14:13:31]

Description was changed from

==========
Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
==========

to

==========
Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
LOG=N
==========

[commit-bot: I haz the power, 2016-04-15 14:29:26]

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

[commit-bot: I haz the power, 2016-04-15 14:29:27]

Try jobs failed on following builders:
  v8_mac_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/230)
  v8_mac_rel_ng_triggered on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng_triggered/bui...)

[nickie, 2016-04-15 14:57:14]

The CQ bit was checked by nikolaos@chromium.org

[nickie, 2016-04-15 14:57:14]

The patchset sent to the CQ was uploaded after l-g-t-m from
rossberg@chromium.org
Link to the patchset: https://codereview.chromium.org/1887743002/#ps20001
(title: "Rebase and mark mjsunit/harmony/regress/regress-4658 as good for
ignition")

[commit-bot: I haz the power, 2016-04-15 14:57:25]

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

[commit-bot: I haz the power, 2016-04-15 18:08:03]

Description was changed from

==========
Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
LOG=N
==========

to

==========
Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
LOG=N
==========

[commit-bot: I haz the power, 2016-04-15 18:08:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-15 18:08:43]

Description was changed from

==========
Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
LOG=N
==========

to

==========
Re-scope inner scopes in arrow parameter initializers

This patch correctly re-scopes inner scopes that can appear in do
expressions used as initializers to arrow parameters.

R=rossberg@chromium.org
BUG=v8:4904
LOG=N

Committed: https://crrev.com/451fa77235e39a494b1abc3bb21a696373397e7e
Cr-Commit-Position: refs/heads/master@{#35542}
==========

[commit-bot: I haz the power, 2016-04-15 18:08:43]

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

[adamk, 2016-04-18 17:25:14]

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

[adamk, 2016-04-18 17:25:14]

Thanks for the fix!