Browse Source

fix: call node::Stop on exit (#25502)

Jeremy Rose 4 years ago
parent
commit
489aafc1f9
2 changed files with 2 additions and 1 deletions
  1. 1 1
      shell/app/node_main.cc
  2. 1 0
      shell/browser/electron_browser_main_parts.cc

+ 1 - 1
shell/app/node_main.cc

@@ -233,7 +233,7 @@ int NodeMain(int argc, char* argv[]) {
 
     node::ResetStdio();
 
-    env->set_can_call_into_js(false);
+    node::Stop(env);
     env->stop_sub_worker_contexts();
     env->RunCleanup();
 

+ 1 - 0
shell/browser/electron_browser_main_parts.cc

@@ -552,6 +552,7 @@ void ElectronBrowserMainParts::PostMainMessageLoopRun() {
   // invoke Node/V8 APIs inside them.
   node_debugger_->Stop();
   node_env_->env()->set_trace_sync_io(false);
+  node::Stop(node_env_->env());
   node_env_.reset();
   js_env_->OnMessageLoopDestroying();