Inline Scope::Initialize into the only constructor that's always called right before

Inline Scope::Initialize into the only constructor that's always called right before

This additionally makes the invariant obvious that outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209
Committed: https://crrev.com/13cbf545223a3166bd8c7d72c2abac4425a0a4a1
Cr-Commit-Position: refs/heads/master@{#37902}

[Toon Verwaest, 2016-07-20 07:55:48]

verwaest@chromium.org changed reviewers:
+ marja@chromium.org, rossberg@chromium.org

[Toon Verwaest, 2016-07-20 07:55:49]

ptal
(rossberg as ast owner, since I'm still not an ast owner myself ... :))

[marja, 2016-07-20 08:50:01]

lgtm % spelling mistake in the description: invariant

[Toon Verwaest, 2016-07-20 11:13:55]

Description was changed from

==========
Inline Scope::Initialize into the only constructor that's always called right
before

This additionally makes the invarient obvious that
outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209
==========

to

==========
Inline Scope::Initialize into the only constructor that's always called right
before

This additionally makes the invariant obvious that
outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209
==========

[Toon Verwaest, 2016-07-20 11:14:01]

The CQ bit was checked by verwaest@chromium.org

[commit-bot: I haz the power, 2016-07-20 11:14:09]

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-07-20 12:16:53]

Description was changed from

==========
Inline Scope::Initialize into the only constructor that's always called right
before

This additionally makes the invariant obvious that
outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209
==========

to

==========
Inline Scope::Initialize into the only constructor that's always called right
before

This additionally makes the invariant obvious that
outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209
==========

[commit-bot: I haz the power, 2016-07-20 12:16:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-20 12:19:56]

Description was changed from

==========
Inline Scope::Initialize into the only constructor that's always called right
before

This additionally makes the invariant obvious that
outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209
==========

to

==========
Inline Scope::Initialize into the only constructor that's always called right
before

This additionally makes the invariant obvious that
outer_scope==nullptr+is_with_scope is impossible.

BUG=v8:5209

Committed: https://crrev.com/13cbf545223a3166bd8c7d72c2abac4425a0a4a1
Cr-Commit-Position: refs/heads/master@{#37902}
==========

[commit-bot: I haz the power, 2016-07-20 12:19:57]

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