Test that "yield" expressions are disallowed in arrow formal parameter initializers

Test that "yield" expressions are disallowed in arrow formal parameter initializers

R=adamk@chromium.org
LOG=N
BUG=v8:4397, v8:4394
Committed: https://crrev.com/5f2d6ef69a52997109f877261b13f46776893474
Cr-Commit-Position: refs/heads/master@{#30459}

[wingo, 2015-08-28 10:04:39]

The late catch in https://codereview.chromium.org/1320673007/ where I added a
"if (*ok == false) return" fixed the failure to enforce the yield restriction
for param initializers of functions that declare 'use strict' in their bodies
instead of outside.

[wingo, 2015-08-28 10:05:25]

wingo@igalia.com changed reviewers:
+ conradw@chromium.org

[wingo, 2015-08-28 10:31:24]

On 2015/08/28 10:04:39, wingo wrote:
> The late catch in https://codereview.chromium.org/1320673007/ where I added a
> "if (*ok == false) return" fixed the failure to enforce the yield restriction
> for param initializers of functions that declare 'use strict' in their bodies
> instead of outside.

Yay, turns out I could add tests for arrow functions too.  This will fix v8:4394
as well..

[wingo, 2015-08-28 10:39:21]

The CQ bit was checked by wingo@igalia.com to run a CQ dry run

[commit-bot: I haz the power, 2015-08-28 10:39:31]

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

[commit-bot: I haz the power, 2015-08-28 11:03:45]

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

[commit-bot: I haz the power, 2015-08-28 11:03:45]

Dry run: This issue passed the CQ dry run.

[caitp (gmail), 2015-08-28 11:18:58]

The cl description refers to something which is no longer legal js (language
mode directive in function with non-simple parameter). Is the message still
correct, or does it need an update?

[wingo, 2015-08-28 11:28:47]

On 2015/08/28 11:18:58, caitp wrote:
> The cl description refers to something which is no longer legal js (language
> mode directive in function with non-simple parameter). Is the message still
> correct, or does it need an update?

Thanks for the note!  I didn't realize that this is now invalid JS:

  function f(x=10) { 'use strict'; }

I actually don't see why the language semantics changed... but OK.  I'll remove
these tests :)

[wingo, 2015-08-28 12:37:19]

Updated description to reflect recent spec change, removed invalid tests.

[wingo, 2015-08-28 12:37:34]

The CQ bit was checked by wingo@igalia.com to run a CQ dry run

[commit-bot: I haz the power, 2015-08-28 12:37:40]

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

[commit-bot: I haz the power, 2015-08-28 13:20:14]

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

[commit-bot: I haz the power, 2015-08-28 13:20:14]

Dry run: This issue passed the CQ dry run.

[adamk, 2015-08-28 15:35:56]

lgtm, but CL description maybe should note that this is a test-only change

[wingo, 2015-08-31 08:23:55]

The CQ bit was checked by wingo@igalia.com

[commit-bot: I haz the power, 2015-08-31 08:24:06]

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

[commit-bot: I haz the power, 2015-08-31 08:47:52]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-08-31 08:47:59]

Patchset 4 (id:??) landed as
https://crrev.com/5f2d6ef69a52997109f877261b13f46776893474
Cr-Commit-Position: refs/heads/master@{#30459}