[es6] Some renamings and minor clean-ups in parameter parsing

[es6] Some renamings and minor clean-ups in parameter parsing

In particular, rename FormalParameterParsingState and friends to FormalParameters etc.

This should not change any logic, but is a preparatory CL for a bunch of follow-up fixes and clean-ups.

R=mstarzinger@chromium.org
BUG=

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

[rossberg, 2015-07-23 11:06:59]

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

[commit-bot: I haz the power, 2015-07-23 11:07:04]

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

[Michael Starzinger, 2015-07-23 11:12:48]

LGTM.

https://codereview.chromium.org/1247443004/diff/1/src/parser.cc
File src/parser.cc (right):

https://codereview.chromium.org/1247443004/diff/1/src/parser.cc#newcode4306
src/parser.cc:4306: if (parameters.has_rest) return false;
This semantic change is intended I assume?

[commit-bot: I haz the power, 2015-07-23 11:17:25]

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

[commit-bot: I haz the power, 2015-07-23 11:17:25]

Dry run: 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/7915)

[rossberg, 2015-07-23 11:21:46]

https://codereview.chromium.org/1247443004/diff/1/src/parser.cc
File src/parser.cc (right):

https://codereview.chromium.org/1247443004/diff/1/src/parser.cc#newcode4306
src/parser.cc:4306: if (parameters.has_rest) return false;
On 2015/07/23 11:12:48, Michael Starzinger wrote:
> This semantic change is intended I assume?

Actually, no. This slipped in when I split up the CL. Thanks for catching!

[rossberg, 2015-07-23 11:21:50]

The CQ bit was checked by rossberg@chromium.org

[rossberg, 2015-07-23 11:21:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
mstarzinger@chromium.org
Link to the patchset: https://codereview.chromium.org/1247443004/#ps20001
(title: "Remove accidental line")

[commit-bot: I haz the power, 2015-07-23 11:21:55]

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

[commit-bot: I haz the power, 2015-07-23 11:53:37]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-23 11:53:55]

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