Skip to content

Commit

Permalink
Remove DisallowJavascriptExecutionScope calls on FreeEnvironment -> u…
Browse files Browse the repository at this point in the history
…v_run stack
  • Loading branch information
blattersturm committed Sep 17, 2021
1 parent 8c64f31 commit 502c055
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
2 changes: 0 additions & 2 deletions src/api/environment.cc
Original file line number Diff line number Diff line change
Expand Up @@ -370,8 +370,6 @@ Environment* CreateEnvironment(

void FreeEnvironment(Environment* env) {
Isolate* isolate = env->isolate();
Isolate::DisallowJavascriptExecutionScope disallow_js(isolate,
Isolate::DisallowJavascriptExecutionScope::THROW_ON_FAILURE);
{
HandleScope handle_scope(isolate); // For env->context().
Context::Scope context_scope(env->context());
Expand Down
3 changes: 0 additions & 3 deletions src/env.cc
Original file line number Diff line number Diff line change
Expand Up @@ -635,9 +635,6 @@ void Environment::CleanupHandles() {
task_queues_async_initialized_ = false;
}

Isolate::DisallowJavascriptExecutionScope disallow_js(isolate(),
Isolate::DisallowJavascriptExecutionScope::THROW_ON_FAILURE);

RunAndClearNativeImmediates(true /* skip unrefed SetImmediate()s */);

for (ReqWrapBase* request : req_wrap_queue_)
Expand Down

0 comments on commit 502c055

Please sign in to comment.