Chengzhong Wu 5db35b4d7d src: avoid draining platform tasks at FreeEnvironment ...

At the point of `FreeEnvironment` and onwards, no JavaScript execution
associated with the Environment should be triggered.

Avoid draining platform tasks that can trigger JavaScript execution in
`FreeEnvironment`. The holder of `node::Environment` should immediately
call `node::MultiIsolatePlatform::UnregisterIsolate` and
`v8::Isolate::Dispose` to cancel pending foreground tasks and join
concurrent tasks after the environment was freed.

`NodePlatform` can properly handle the case in `RunForegroundTask` when
an Isolate out-lives its associated `node::Environment`.

PR-URL: https://github.com/nodejs/node/pull/51290
Fixes: https://github.com/nodejs/node/issues/47748
Fixes: https://github.com/nodejs/node/issues/49344
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>

2024-01-08 14:39:30 +00:00

..

async_resource.cc

src: add a constructor overload for CallbackScope

2021-08-21 06:53:50 +00:00

callback.cc

src: refactor GetCreationContext calls

2023-12-28 11:12:34 +00:00

embed_helpers.cc

src: support configurable snapshot

2023-12-15 15:13:46 +01:00

encoding.cc

test: add more cases for parse-encoding

2022-09-06 06:20:16 +00:00

environment.cc

src: avoid draining platform tasks at FreeEnvironment

2024-01-08 14:39:30 +00:00

exceptions.cc

src: add and fix some preprocessor comments

2021-11-10 15:24:22 +01:00

hooks.cc

src: simplify exit code accesses

2022-11-10 01:09:18 +00:00

utils.cc

src: cleanup unused headers

2019-11-28 23:21:08 +01:00