(Pre)Parser: Simplify NewExpression handling.

(Pre)Parser: Simplify NewExpression handling.

Notes:
- We use simple recursion to keep track of how many "new" operators we have seen
  and where.
- This makes the self-baked stack class PositionStack in parser.cc unnecessary.
- Now the logic is also unified between Parser and PreParser.
- It might have been a copy-paste artifact (ParseLeftHandSideExpression ->
  ParseMemberWithNewPrefixesExpression) that the logic was so complicated
  before.

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

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

[marja, 2014-02-14 15:08:33]

ulan, ptal

[ulan, 2014-02-14 15:20:03]

Good stuff, LGTM!

[marja, 2014-02-14 15:33:22]

Committed patchset #1 manually as r19386 (presubmit successful).

[marja, 2014-02-17 15:11:42]

https://codereview.chromium.org/166943002/diff/1/src/parser.cc
File src/parser.cc (left):

https://codereview.chromium.org/166943002/diff/1/src/parser.cc#oldcode3477
src/parser.cc:3477: result = factory()->NewCallNew(result, args, pos);
The problem is:

In the original code, we can have "new foo()" and this loop will continue
parsing the . and [ parts, so, "new foo().bar" gets parsed properly.

In the new code, we no longer loop the . and [ parts once we have found () and
constructed the NewExpression.