Move function length tracking from Scope to (Pre)?ParserFormalParameters.

Move function length tracking from Scope to (Pre)?ParserFormalParameters.

It belongs there more logically. In addition, this is a pre-step needed
for preparsing the parameters of a preparsed function.

In addition, move the "subtract rest parameter from arity" logic from
Parser to (Pre)?ParserFormalParameters.

BUG=v8:5515
Committed: https://crrev.com/a726e85fe33e25ed057d76b6b8027ca6094ed9e5
Cr-Commit-Position: refs/heads/master@{#40258}

[marja, 2016-10-13 11:57:31]

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

[commit-bot: I haz the power, 2016-10-13 11:57:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Toon Verwaest, 2016-10-13 12:05:58]

verwaest@chromium.org changed reviewers:
+ verwaest@chromium.org

[Toon Verwaest, 2016-10-13 12:05:59]

Very nice cleanup! lgtm. Just remove arity_ from DeclarationScope :)

[marja, 2016-10-13 12:06:43]

The CQ bit was checked by marja@chromium.org

[marja, 2016-10-13 12:06:44]

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

[commit-bot: I haz the power, 2016-10-13 12:06:47]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-13 12:34:42]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-13 12:35:04]

Description was changed from

==========
Move function length tracking from Scope to (Pre)?ParserFormalParameters.

It belongs there more logically. In addition, this is a pre-step needed
for preparsing the parameters of a preparsed function.

In addition, move the "subtract rest parameter from arity" logic from
Parser to (Pre)?ParserFormalParameters.

BUG=v8:5515
==========

to

==========
Move function length tracking from Scope to (Pre)?ParserFormalParameters.

It belongs there more logically. In addition, this is a pre-step needed
for preparsing the parameters of a preparsed function.

In addition, move the "subtract rest parameter from arity" logic from
Parser to (Pre)?ParserFormalParameters.

BUG=v8:5515

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

[commit-bot: I haz the power, 2016-10-13 12:35:05]

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