Revert of Always deserialize scope infos for parsing

Revert of Always deserialize scope infos for parsing (patchset #3 id:40001 of https://codereview.chromium.org/2280933002/ )

Reason for revert:
Significantly tanks parsing. We probably should just keep on doing what we're doing: partially deserialize while resolving variables. If we do scope-info backed resolution after regular resolution based on remaining free variables, we can probably reduce the time-frame of that part. We soon after anyway need to sync with the main thread.

Original issue's description:
> Always deserialize scope infos for parsing
>
> When looking up variables in the ScopeInfo, we did a linear scan of the
> ScopeInfo. Since that's unacceptably slow, a context slot cache was added
> that would speed up repeated lookups of the same variable.
>
> Instead, just always fully convert the ScopeInfo into scopes, so they can
> lookup variables without scanning the ScopeInfo.
>
> This also allows for removing the now unused ContextSlotCache.
>
> R=adamk@chromium.org,verwaest@chromium.org,marja@chromium.org
> BUG=v8:5315
>
> Committed: https://crrev.com/81f824cad18e4dc873a8838943217eb9c9f0c1f0
> Cr-Commit-Position: refs/heads/master@{#38953}

TBR=adamk@chromium.org,marja@chromium.org,jochen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5315
Committed: https://crrev.com/f046cb95439ca501bae8dc5eed0dbefa87d7f148
Cr-Commit-Position: refs/heads/master@{#38958}

[Toon Verwaest, 2016-08-27 07:08:11]

The CQ bit was checked by verwaest@chromium.org

[Toon Verwaest, 2016-08-27 07:08:11]

Created Revert of Always deserialize scope infos for parsing

[commit-bot: I haz the power, 2016-08-27 07:08:19]

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-08-27 07:08:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-27 07:09:09]

Description was changed from

==========
Revert of Always deserialize scope infos for parsing (patchset #3 id:40001 of
https://codereview.chromium.org/2280933002/ )

Reason for revert:
Significantly tanks parsing. We probably should just keep on doing what we're
doing: partially deserialize while resolving variables. If we do scope-info
backed resolution after regular resolution based on remaining free variables, we
can probably reduce the time-frame of that part. We soon after anyway need to
sync with the main thread.

Original issue's description:
> Always deserialize scope infos for parsing
> 
> When looking up variables in the ScopeInfo, we did a linear scan of the
> ScopeInfo. Since that's unacceptably slow, a context slot cache was added
> that would speed up repeated lookups of the same variable.
> 
> Instead, just always fully convert the ScopeInfo into scopes, so they can
> lookup variables without scanning the ScopeInfo.
> 
> This also allows for removing the now unused ContextSlotCache.
> 
> R=adamk@chromium.org,verwaest@chromium.org,marja@chromium.org
> BUG=v8:5315
> 
> Committed: https://crrev.com/81f824cad18e4dc873a8838943217eb9c9f0c1f0
> Cr-Commit-Position: refs/heads/master@{#38953}

TBR=adamk@chromium.org,marja@chromium.org,jochen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5315
==========

to

==========
Revert of Always deserialize scope infos for parsing (patchset #3 id:40001 of
https://codereview.chromium.org/2280933002/ )

Reason for revert:
Significantly tanks parsing. We probably should just keep on doing what we're
doing: partially deserialize while resolving variables. If we do scope-info
backed resolution after regular resolution based on remaining free variables, we
can probably reduce the time-frame of that part. We soon after anyway need to
sync with the main thread.

Original issue's description:
> Always deserialize scope infos for parsing
>
> When looking up variables in the ScopeInfo, we did a linear scan of the
> ScopeInfo. Since that's unacceptably slow, a context slot cache was added
> that would speed up repeated lookups of the same variable.
>
> Instead, just always fully convert the ScopeInfo into scopes, so they can
> lookup variables without scanning the ScopeInfo.
>
> This also allows for removing the now unused ContextSlotCache.
>
> R=adamk@chromium.org,verwaest@chromium.org,marja@chromium.org
> BUG=v8:5315
>
> Committed: https://crrev.com/81f824cad18e4dc873a8838943217eb9c9f0c1f0
> Cr-Commit-Position: refs/heads/master@{#38953}

TBR=adamk@chromium.org,marja@chromium.org,jochen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=v8:5315

Committed: https://crrev.com/f046cb95439ca501bae8dc5eed0dbefa87d7f148
Cr-Commit-Position: refs/heads/master@{#38958}
==========

[commit-bot: I haz the power, 2016-08-27 07:09:10]

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