Move ParseArguments to ParserBase and add tests.

Move ParseArguments to ParserBase and add tests.

Notes:
- PreParser didn't produce "too_many_arguments"; now it does.
- The argument count in the error message was wrong; fixed it.

BUG=v8:3126
LOG=N
R=ulan@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=19812

[marja, 2014-03-11 13:42:58]

ulan, pls review this small one too.

[ulan, 2014-03-11 15:39:58]

lgtm

https://codereview.chromium.org/194503004/diff/20001/src/preparser.h
File src/preparser.h (right):

https://codereview.chromium.org/194503004/diff/20001/src/preparser.h#newcode1503
src/preparser.h:1503: if (!done) Expect(Token::COMMA,
CHECK_OK_CUSTOM(NullExpressionList));
As discussed offline, it is probably worth putting Expect in {}

[marja, 2014-03-11 15:49:10]

thx for review & good catch!

https://codereview.chromium.org/194503004/diff/20001/src/preparser.h
File src/preparser.h (right):

https://codereview.chromium.org/194503004/diff/20001/src/preparser.h#newcode1503
src/preparser.h:1503: if (!done) Expect(Token::COMMA,
CHECK_OK_CUSTOM(NullExpressionList));
On 2014/03/11 15:39:59, ulan wrote:
> As discussed offline, it is probably worth putting Expect in {}

Done.

[marja, 2014-03-11 16:30:57]

Committed patchset #4 manually as r19812 (presubmit successful).