Ensure arrow functions can close over lexically-scoped variables

Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934, v8:4255
LOG=y

Committed: https://crrev.com/953bb416a3675a4bd899e01625b8b584480e488a
Cr-Commit-Position: refs/heads/master@{#33530}

[adamk, 2016-01-26 22:44:38]

Description was changed from

==========
Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934
LOG=y
==========

to

==========
Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934, v8:4255
LOG=y
==========

[adamk, 2016-01-26 22:45:40]

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

[adamk, 2016-01-26 22:45:41]

This check seems to have been missed when Yang fixed block scoping in
https://codereview.chromium.org/1216013002

[rossberg, 2016-01-26 22:55:51]

lgtm

[adamk, 2016-01-26 22:58:02]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-01-26 22:58:06]

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

[commit-bot: I haz the power, 2016-01-26 23:10:27]

Description was changed from

==========
Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934, v8:4255
LOG=y
==========

to

==========
Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934, v8:4255
LOG=y
==========

[commit-bot: I haz the power, 2016-01-26 23:10:28]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-01-26 23:11:18]

Description was changed from

==========
Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934, v8:4255
LOG=y
==========

to

==========
Ensure arrow functions can close over lexically-scoped variables

ParseArrowFunctionLiteral was erroneously checking AllowsLazyCompilation
rather than AllowsLazyParsing when deciding whether to parse lazily.
This meant that lexically-scoped variables that had no other referents
wouldn't get closed over properly.

BUG=chromium:580934, v8:4255
LOG=y

Committed: https://crrev.com/953bb416a3675a4bd899e01625b8b584480e488a
Cr-Commit-Position: refs/heads/master@{#33530}
==========

[commit-bot: I haz the power, 2016-01-26 23:11:19]

Patchset 1 (id:??) landed as
https://crrev.com/953bb416a3675a4bd899e01625b8b584480e488a
Cr-Commit-Position: refs/heads/master@{#33530}