Remove unused 'needs_init' member of ParsingResult

Remove unused 'needs_init' member of ParsingResult

Also various related cleanup in ParseVariableDeclarations(). The only
changes in logic are explained below:

  - We were redundantly checking for parenthesized binding patterns;
    these are already ruled out by BindingPatternUnexpectedToken()
    calls in the places where we hit an LPAREN.
  - There's no need to default-initialize a LET-mode variable in a
    for-each loop, just as there isn't for CONST or CONST_LEGACY
    (ParseForStatement will take care of properly initializing all
    of the above).

Committed: https://crrev.com/3d56b0d7c03690627a74125869636e621b7ae8dd
Cr-Commit-Position: refs/heads/master@{#33749}

[adamk, 2016-02-03 23:32:02]

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

[adamk, 2016-02-04 00:14:07]

This patch also caused me to question whether we really needed the
is_parenthesized bit, and https://codereview.chromium.org/1665043002/ suggests
the answer is "no" (unless I'm missing something).

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

Nice, lgtm.

[adamk, 2016-02-04 18:42:18]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2016-02-04 18:42:27]

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

[commit-bot: I haz the power, 2016-02-04 18:44:01]

Committed patchset #1 (id:1)

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

Description was changed from

==========
Remove unused 'needs_init' member of ParsingResult

Also various related cleanup in ParseVariableDeclarations(). The only
changes in logic are explained below:

  - We were redundantly checking for parenthesized binding patterns;
    these are already ruled out by BindingPatternUnexpectedToken()
    calls in the places where we hit an LPAREN.
  - There's no need to default-initialize a LET-mode variable in a
    for-each loop, just as there isn't for CONST or CONST_LEGACY
    (ParseForStatement will take care of properly initializing all
    of the above).
==========

to

==========
Remove unused 'needs_init' member of ParsingResult

Also various related cleanup in ParseVariableDeclarations(). The only
changes in logic are explained below:

  - We were redundantly checking for parenthesized binding patterns;
    these are already ruled out by BindingPatternUnexpectedToken()
    calls in the places where we hit an LPAREN.
  - There's no need to default-initialize a LET-mode variable in a
    for-each loop, just as there isn't for CONST or CONST_LEGACY
    (ParseForStatement will take care of properly initializing all
    of the above).

Committed: https://crrev.com/3d56b0d7c03690627a74125869636e621b7ae8dd
Cr-Commit-Position: refs/heads/master@{#33749}
==========

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

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