Slight cleanup of TryCatch parsing/variable declaration

Slight cleanup of TryCatch parsing/variable declaration

Instead of unconditionally parsing the catch parameter as an expression
and then recovering if it turns out to be a simple variable proxy
(the overwhelmingly common case), this patch peeks one token ahead
before attempting to parse. This avoids doing the usual RemoveUnresolved
gymnastics in ParseTryStatement, and as a side-effect slightly improves
function name inference for an async arrow function test case.

Committed: https://crrev.com/458bd3b9a3ee150a19d729a6970065b5ec0a0e17
Cr-Commit-Position: refs/heads/master@{#37780}

[adamk, 2016-07-13 21:36:24]

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

[commit-bot: I haz the power, 2016-07-13 21:36:30]

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

[adamk, 2016-07-13 21:37:20]

adamk@chromium.org changed reviewers:
+ neis@chromium.org

[commit-bot: I haz the power, 2016-07-13 22:00:20]

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

[commit-bot: I haz the power, 2016-07-13 22:00:21]

Dry run: This issue passed the CQ dry run.

[neis, 2016-07-14 08:14:13]

Nice! lgtm

[adamk, 2016-07-14 17:19:23]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-07-14 17:19:28]

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-07-14 17:52:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-14 17:52:41]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-14 17:55:49]

Description was changed from

==========
Slight cleanup of TryCatch parsing/variable declaration

Instead of unconditionally parsing the catch parameter as an expression
and then recovering if it turns out to be a simple variable proxy
(the overwhelmingly common case), this patch peeks one token ahead
before attempting to parse. This avoids doing the usual RemoveUnresolved
gymnastics in ParseTryStatement, and as a side-effect slightly improves
function name inference for an async arrow function test case.
==========

to

==========
Slight cleanup of TryCatch parsing/variable declaration

Instead of unconditionally parsing the catch parameter as an expression
and then recovering if it turns out to be a simple variable proxy
(the overwhelmingly common case), this patch peeks one token ahead
before attempting to parse. This avoids doing the usual RemoveUnresolved
gymnastics in ParseTryStatement, and as a side-effect slightly improves
function name inference for an async arrow function test case.

Committed: https://crrev.com/458bd3b9a3ee150a19d729a6970065b5ec0a0e17
Cr-Commit-Position: refs/heads/master@{#37780}
==========

[commit-bot: I haz the power, 2016-07-14 17:55:49]

Patchset 1 (id:??) landed as
https://crrev.com/458bd3b9a3ee150a19d729a6970065b5ec0a0e17
Cr-Commit-Position: refs/heads/master@{#37780}