PreParsing inner functions: Fix declaration-only variables.

PreParsing inner functions: Fix declaration-only variables.

If an inner function only declares a variable but doesn't use it, Parser
and PreParser produced different unresolved variables, and that confused
the pessimistic context allocation.

BUG=chromium:650969
Committed: https://crrev.com/22ff09e06a3c4e26e831be228355e93b81bd3ab1
Cr-Commit-Position: refs/heads/master@{#39947}

[marja, 2016-10-04 08:35:59]

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

[commit-bot: I haz the power, 2016-10-04 08:36:04]

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

[Toon Verwaest, 2016-10-04 09:10:04]

verwaest@chromium.org changed reviewers:
+ verwaest@chromium.org

[Toon Verwaest, 2016-10-04 09:10:04]

lgtm, but keep off the flag for now.

[marja, 2016-10-04 09:11:41]

Yep, as discussed offline, keeping the flag off until I've fixed the overly
eager context allocation; this CL doesn't yet fix it.

[marja, 2016-10-04 09:12:01]

The CQ bit was checked by marja@chromium.org

[marja, 2016-10-04 09:12:02]

The patchset sent to the CQ was uploaded after l-g-t-m from
verwaest@chromium.org
Link to the patchset: https://codereview.chromium.org/2388183003/#ps60001
(title: "flag off")

[commit-bot: I haz the power, 2016-10-04 09:12:04]

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-10-04 09:38:31]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-10-04 09:38:55]

Description was changed from

==========
PreParsing inner functions: Fix declaration-only variables.

If an inner function only declares a variable but doesn't use it, Parser
and PreParser produced different unresolved variables, and that confused
the pessimistic context allocation.

BUG=chromium:650969
==========

to

==========
PreParsing inner functions: Fix declaration-only variables.

If an inner function only declares a variable but doesn't use it, Parser
and PreParser produced different unresolved variables, and that confused
the pessimistic context allocation.

BUG=chromium:650969

Committed: https://crrev.com/22ff09e06a3c4e26e831be228355e93b81bd3ab1
Cr-Commit-Position: refs/heads/master@{#39947}
==========

[commit-bot: I haz the power, 2016-10-04 09:38:56]

Patchset 4 (id:??) landed as
https://crrev.com/22ff09e06a3c4e26e831be228355e93b81bd3ab1
Cr-Commit-Position: refs/heads/master@{#39947}