Ship ES2015 restrictions on function declaration locations

Ship ES2015 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2015 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel

Committed: https://crrev.com/f9db79ebdb5ab2dd559b5070d905332e489c15ec
Cr-Commit-Position: refs/heads/master@{#34828}

[Dan Ehrenberg, 2016-03-15 00:56:28]

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

[commit-bot: I haz the power, 2016-03-15 00:56:33]

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

[commit-bot: I haz the power, 2016-03-15 01:13:44]

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

[commit-bot: I haz the power, 2016-03-15 01:13:45]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel/builds/...)

[Dan Ehrenberg, 2016-03-15 23:48:07]

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

[commit-bot: I haz the power, 2016-03-15 23:48:16]

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

[commit-bot: I haz the power, 2016-03-15 23:49:26]

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

[commit-bot: I haz the power, 2016-03-15 23:49:27]

Dry run: Try jobs failed on following builders:
  v8_linux64_asan_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_asan_rel/builds/...)
  v8_linux64_avx2_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel/builds/...)
  v8_linux64_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_rel_ng/builds/2976)
  v8_linux_arm64_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel/builds/1...)
  v8_linux_arm_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel/builds/15391)
  v8_linux_nodcheck_rel on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel/build...)
  v8_linux_rel_ng on tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/2988)

[Dan Ehrenberg, 2016-03-16 00:49:36]

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

[commit-bot: I haz the power, 2016-03-16 00:49:45]

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

[Dan Ehrenberg, 2016-03-16 01:15:20]

Description was changed from

==========
Ship ES2017 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2017 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
==========

to

==========
Ship ES2017 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2017 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-03-16 02:40:25]

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

[commit-bot: I haz the power, 2016-03-16 02:40:26]

Dry run: This issue passed the CQ dry run.

[adamk, 2016-03-16 16:08:59]

lgtm

These restrictions were also present in ES2015, right? I find referring to it as
"ES2017" confusing if so.

[Dan Ehrenberg, 2016-03-16 16:37:08]

Description was changed from

==========
Ship ES2017 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2017 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

to

==========
Ship ES2015 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2015 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

[Dan Ehrenberg, 2016-03-16 16:37:10]

The CQ bit was checked by littledan@chromium.org

[commit-bot: I haz the power, 2016-03-16 16:37:13]

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

[commit-bot: I haz the power, 2016-03-16 16:39:14]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-16 16:39:32]

Description was changed from

==========
Ship ES2015 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2015 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel
==========

to

==========
Ship ES2015 restrictions on function declaration locations

Function declarations were previously permitted by V8 in many locations
which no ECMAScript specification allowed; the ECMAScript 2015 spec
enumerates a few locations (in blocks, as well as after labels and in
conditionals when in sloppy mode). This patch ships the flag to restrict
the usage of function declarations to those contexts.

R=adamk
LOG=Y
BUG=v8:4824
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_chromium_rel_ng;tryserver.blink:linux_blink_rel

Committed: https://crrev.com/f9db79ebdb5ab2dd559b5070d905332e489c15ec
Cr-Commit-Position: refs/heads/master@{#34828}
==========

[commit-bot: I haz the power, 2016-03-16 16:39:33]

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