Chromium Code Reviews| Index: src/runtime/runtime-scopes.cc |
| diff --git a/src/runtime/runtime-scopes.cc b/src/runtime/runtime-scopes.cc |
| index 0c037db307a60ca994da643e9a2554b462654ced..e6de242b227f8afc3ac4e8ed282b980ff7535c2a 100644 |
| --- a/src/runtime/runtime-scopes.cc |
| +++ b/src/runtime/runtime-scopes.cc |
| @@ -232,7 +232,7 @@ Object* DeclareEvalHelper(Isolate* isolate, Handle<String> name, |
| Handle<Context> context(context_arg->declaration_context(), isolate); |
| DCHECK(context->IsFunctionContext() || context->IsNativeContext() || |
| - context->IsScriptContext() || |
| + context->IsScriptContext() || context->IsEvalContext() || |
| (context->IsBlockContext() && context->has_extension())); |
| bool is_function = value->IsJSFunction(); |
| @@ -313,7 +313,7 @@ Object* DeclareEvalHelper(Isolate* isolate, Handle<String> name, |
| } |
| DCHECK(object->IsJSContextExtensionObject() || object->IsJSGlobalObject()); |
| } else { |
| - DCHECK(context->IsFunctionContext()); |
| + DCHECK(context->IsFunctionContext() || context->IsEvalContext()); |
|
adamk
2016/11/12 00:26:05
We should never get here for an eval context, sinc
Dan Ehrenberg
2016/12/07 05:41:26
Reverted and added a comment explaining the DCHECK
|
| object = |
| isolate->factory()->NewJSObject(isolate->context_extension_function()); |
| context->set_extension(*object); |
| @@ -696,6 +696,16 @@ RUNTIME_FUNCTION(Runtime_NewFunctionContext) { |
| return *isolate->factory()->NewFunctionContext(length, function); |
| } |
| +RUNTIME_FUNCTION(Runtime_NewEvalContext) { |
| + HandleScope scope(isolate); |
| + DCHECK(args.length() == 1); |
| + |
| + CONVERT_ARG_HANDLE_CHECKED(JSFunction, function, 0); |
| + |
| + DCHECK(function->context() == isolate->context()); |
| + int length = function->shared()->scope_info()->ContextLength(); |
| + return *isolate->factory()->NewEvalContext(length, function); |
| +} |
| RUNTIME_FUNCTION(Runtime_PushWithContext) { |
| HandleScope scope(isolate); |