Unify (Pre)Parser::ParseTryStatement.

Unify (Pre)Parser::ParseTryStatement.

Notes:
- This makes Parser and PreParser produce the same errors with the added test
cases (this was not the case before).
- ParseBlock already does Expect(Token::LBRACE), so no need to check it twice.

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

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

[Michael Starzinger, 2014-02-07 14:59:59]

LGTM with one suggestion.

https://codereview.chromium.org/148233011/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/148233011/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:1994: RunParserSyncTest(context_data,
statement_data, kError);
Can we also test the one or two success cases, just for the heck of it?

[marja, 2014-02-07 15:07:55]

thanks for review!

https://codereview.chromium.org/148233011/diff/1/test/cctest/test-parsing.cc
File test/cctest/test-parsing.cc (right):

https://codereview.chromium.org/148233011/diff/1/test/cctest/test-parsing.cc#...
test/cctest/test-parsing.cc:1994: RunParserSyncTest(context_data,
statement_data, kError);
On 2014/02/07 15:00:00, Michael Starzinger wrote:
> Can we also test the one or two success cases, just for the heck of it?

Done.

[marja, 2014-02-10 08:45:19]

Committed patchset #2 manually as r19212 (presubmit successful).