Always give class literals a block scope

Always give class literals a block scope

Re-land of https://crrev.com/cf13dda1ba25e8293ea143f33c6c5f6233a39c86,
fixing the issue with vector stores.

Class methods always have the class scope on their scope chain in order
to implement strong mode checks. Previously, that scope wasn't attached
to the ClassLiteral for anonymous classes (since the scope contained
no bindings).

This patch simply puts that same scope on the ClassLiteral, anonymous
or not, which simplifies other code that needs to reason about the scope
of a class and its methods.

Committed: https://crrev.com/b065411f81e1fff769de691d8494d6f75c7132a9
Cr-Commit-Position: refs/heads/master@{#31381}

[adamk, 2015-10-19 13:32:05]

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

[adamk, 2015-10-19 13:33:37]

Description was changed from

==========
Always give class literals a block scope

Class methods always have the class scope on their scope chain in order
to implement strong mode checks. Previously, that scope wasn't attached
to the ClassLiteral for anonymous classes (since the scope contained
no bindings).

This patch simply puts that same scope on the ClassLiteral, anonymous
or not, which simplifies other code that needs to reason about the scope
of a class and its methods.

Fixes the vector-stores code to handle a never-null scope correctly.
==========

to

==========
Always give class literals a block scope

Re-land of https://crrev.com/cf13dda1ba25e8293ea143f33c6c5f6233a39c86,
fixing the issue with vector stores.

Class methods always have the class scope on their scope chain in order
to implement strong mode checks. Previously, that scope wasn't attached
to the ClassLiteral for anonymous classes (since the scope contained
no bindings).

This patch simply puts that same scope on the ClassLiteral, anonymous
or not, which simplifies other code that needs to reason about the scope
of a class and its methods.
==========

[Michael Starzinger, 2015-10-19 15:11:29]

LGTM on patch set #2.

[adamk, 2015-10-19 15:14:45]

The CQ bit was checked by adamk@chromium.org

[commit-bot: I haz the power, 2015-10-19 15:14:50]

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

[commit-bot: I haz the power, 2015-10-19 15:16:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-10-19 15:16:33]

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