[es6] Use strict arguments objects for destructured parameters

[es6] Use strict arguments objects for destructured parameters

Plus some renaming for consistency.

R=adamk@chromium.org
BUG=v8:811
LOG=N

Committed: https://crrev.com/826f8da55fb868a365d047a4a653eb8ff2bfc14e
Cr-Commit-Position: refs/heads/master@{#30064}

[adamk, 2015-08-06 17:12:43]

Can you add some tests with arrow functions? Want to make sure that's covered,
as some of the logic upstream is different (though should all flow through the
same place).

[adamk, 2015-08-06 18:41:33]

lgtm with added tests, though

[rossberg, 2015-08-06 18:56:29]

On 2015/08/06 17:12:43, adamk wrote:
> Can you add some tests with arrow functions? Want to make sure that's covered,
> as some of the logic upstream is different (though should all flow through the
> same place).

That's funny: I actually had tests for arrow functions, and then spent an hour
today trying to figure out why they don't "work". Until I realised that they
(and me) were stupid, because arrows don't bind `arguments`. :)

[adamk, 2015-08-06 18:59:03]

On 2015/08/06 18:56:29, rossberg wrote:
> On 2015/08/06 17:12:43, adamk wrote:
> > Can you add some tests with arrow functions? Want to make sure that's
covered,
> > as some of the logic upstream is different (though should all flow through
the
> > same place).
> 
> That's funny: I actually had tests for arrow functions, and then spent an hour
> today trying to figure out why they don't "work". Until I realised that they
> (and me) were stupid, because arrows don't bind `arguments`. :)

Ha! Some language experts we are...

[rossberg, 2015-08-06 20:26:02]

The CQ bit was checked by rossberg@chromium.org

[commit-bot: I haz the power, 2015-08-06 20:26:18]

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

[commit-bot: I haz the power, 2015-08-06 20:40:57]

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

[commit-bot: I haz the power, 2015-08-06 20:40:57]

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

[rossberg, 2015-08-07 11:13:13]

The CQ bit was checked by rossberg@chromium.org

[rossberg, 2015-08-07 11:13:13]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/1278783002/#ps40001
(title: "Doh")

[commit-bot: I haz the power, 2015-08-07 11:13:28]

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

[commit-bot: I haz the power, 2015-08-07 11:38:29]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-08-07 11:38:58]

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