Remove redundant ParseInfo::scope_.

Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

(Additional change: mark getters as const.)

R=adamk@chromium.org
BUG=

Committed: https://crrev.com/23ea0782977ed3a4dd113462af9ecbfd6ff0ce94
Cr-Commit-Position: refs/heads/master@{#38372}

[neis, 2016-08-04 13:11:18]

Description was changed from

==========
Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

R=adamk@chromium.org
BUG=
==========

to

==========
Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

(Additional change: mark getters as const.)

R=adamk@chromium.org
BUG=
==========

[adamk, 2016-08-04 17:39:32]

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

[adamk, 2016-08-04 17:39:32]

lgtm, but +marja with whom I just this morning had a discussion about
FunctionLiteral::scope_. I think this is still fine, but I want to make sure
this doesn't upset her plans.

[adamk, 2016-08-04 21:23:04]

adamk@chromium.org changed reviewers:
- marja@chromium.org

[adamk, 2016-08-04 21:23:04]

Marja ended up not making the proposed change, so I think this is good to go
(marja to CC).

[marja, 2016-08-05 07:03:51]

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

[marja, 2016-08-05 07:03:52]

Yep, lgtm, as I didn't end up putting null Scopes to FunctionLiteral.

[neis, 2016-08-05 10:56:16]

The CQ bit was checked by neis@chromium.org

[neis, 2016-08-05 10:56:16]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org,
marja@chromium.org
Link to the patchset: https://codereview.chromium.org/2216563003/#ps60001
(title: "Rebase.")

[commit-bot: I haz the power, 2016-08-05 10:56:25]

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-05 11:17:14]

Description was changed from

==========
Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

(Additional change: mark getters as const.)

R=adamk@chromium.org
BUG=
==========

to

==========
Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

(Additional change: mark getters as const.)

R=adamk@chromium.org
BUG=
==========

[commit-bot: I haz the power, 2016-08-05 11:17:15]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-05 11:17:56]

Description was changed from

==========
Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

(Additional change: mark getters as const.)

R=adamk@chromium.org
BUG=
==========

to

==========
Remove redundant ParseInfo::scope_.

This was always set to the literal's scope.

(Additional change: mark getters as const.)

R=adamk@chromium.org
BUG=

Committed: https://crrev.com/23ea0782977ed3a4dd113462af9ecbfd6ff0ce94
Cr-Commit-Position: refs/heads/master@{#38372}
==========

[commit-bot: I haz the power, 2016-08-05 11:17:58]

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

[Michael Achenbach, 2016-08-05 12:20:26]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/2222503002/ by machenbach@chromium.org.

The reason for reverting is: Reverting to revert
https://codereview.chromium.org/2209573002.

[Michael Achenbach, 2016-08-05 12:21:14]

Please just reland a rebased version. Sorry for the inconvenience.