Remove is_parenthesized bit from Expression and PreParserExpression

Remove is_parenthesized bit from Expression and PreParserExpression

This bit was ostensibly being used to provide appropriate syntax
errors for invalid destructuring assignment patterns, but adding a
single call to RecordPatternError() (in place of
BindingPatternUnexpectedToken()) seems to have replaced the need for it.

Committed: https://crrev.com/1c318a9e4c5d379f78e48062e7a1b0be122d6f56
Cr-Commit-Position: refs/heads/master@{#33750}

[adamk, 2016-02-04 00:09:37]

adamk@chromium.org changed reviewers:
+ caitpotter88@gmail.com, rossberg@chromium.org

[adamk, 2016-02-04 00:09:37]

Caitlin, am I missing something here? This appears to pass all tests...

[caitp (gmail), 2016-02-04 02:02:23]

On 2016/02/04 00:09:37, adamk wrote:
> Caitlin, am I missing something here? This appears to pass all tests...

That's strange, because this patch doesn't even build for me locally on top of
master --- `is_parenthesized()` is referenced in preparser.cc and parser.cc

[caitp (gmail), 2016-02-04 02:03:32]

On 2016/02/04 02:02:23, caitp wrote:
> On 2016/02/04 00:09:37, adamk wrote:
> > Caitlin, am I missing something here? This appears to pass all tests...
> 
> That's strange, because this patch doesn't even build for me locally on top of
> master --- `is_parenthesized()` is referenced in preparser.cc and parser.cc

ah nevermind, applying the other patch too, lets see..

[caitp (gmail), 2016-02-04 02:12:27]

lgtm

[rossberg, 2016-02-04 13:33:29]

LGTM. A left-over from the pre-ExpressionClassifier days.

[adamk, 2016-02-04 18:52:10]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-02-04 18:52:17]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1665043002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1665043002/1

[commit-bot: I haz the power, 2016-02-04 18:53:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-04 18:54:37]

Description was changed from

==========
Remove is_parenthesized bit from Expression and PreParserExpression

This bit was ostensibly being used to provide appropriate syntax
errors for invalid destructuring assignment patterns, but adding a
single call to RecordPatternError() (in place of
BindingPatternUnexpectedToken()) seems to have replaced the need for it.
==========

to

==========
Remove is_parenthesized bit from Expression and PreParserExpression

This bit was ostensibly being used to provide appropriate syntax
errors for invalid destructuring assignment patterns, but adding a
single call to RecordPatternError() (in place of
BindingPatternUnexpectedToken()) seems to have replaced the need for it.

Committed: https://crrev.com/1c318a9e4c5d379f78e48062e7a1b0be122d6f56
Cr-Commit-Position: refs/heads/master@{#33750}
==========

[commit-bot: I haz the power, 2016-02-04 18:54:38]

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