[es6] Fix function context check for super and new.target

[es6] Fix function context check for super and new.target

R=adamk@chromium.org
BUG=v8:3330
LOG=N

Committed: https://crrev.com/47d3bb1c0b294b26e97a70988c10c756d266af84
Cr-Commit-Position: refs/heads/master@{#29803}

[adamk, 2015-07-22 20:55:50]

lgtm % comment nit (which can happen whenever, really).

https://codereview.chromium.org/1244423003/diff/1/src/scopes.h
File src/scopes.h (right):

https://codereview.chromium.org/1244423003/diff/1/src/scopes.h#newcode486
src/scopes.h:486: // Find the first function, script, or eval scope.  This is
the scope
Can you fully fix this comment while you're here to include the declaration
block scopes?

[rossberg, 2015-07-23 09:33:58]

Also added a test for super calls.

https://codereview.chromium.org/1244423003/diff/1/src/scopes.h
File src/scopes.h (right):

https://codereview.chromium.org/1244423003/diff/1/src/scopes.h#newcode486
src/scopes.h:486: // Find the first function, script, or eval scope.  This is
the scope
On 2015/07/22 20:55:50, adamk wrote:
> Can you fully fix this comment while you're here to include the declaration
> block scopes?

Done.

[rossberg, 2015-07-23 09:34:03]

The CQ bit was checked by rossberg@chromium.org

[rossberg, 2015-07-23 09:34:04]

The patchset sent to the CQ was uploaded after l-g-t-m from adamk@chromium.org
Link to the patchset: https://codereview.chromium.org/1244423003/#ps20001
(title: "Comment; added test for super calls")

[commit-bot: I haz the power, 2015-07-23 09:34:15]

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

[commit-bot: I haz the power, 2015-07-23 09:58:41]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-23 09:59:11]

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