From 7f85c85b7152d312e89daa68b2d59e9b0d61c5d7 Mon Sep 17 00:00:00 2001 From: "trop[bot]" <37223003+trop[bot]@users.noreply.github.com> Date: Sat, 22 Mar 2025 10:44:04 -0500 Subject: [PATCH] fix: add missing `cpp_heap` to Node.js worker `CreateParams` (#46176) fix: add missing cpp_heap to Node.js worker CreateParams Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Shelley Vohr --- ..._attach_cppgc_heap_on_v8_isolate_creation.patch | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/patches/node/refactor_attach_cppgc_heap_on_v8_isolate_creation.patch b/patches/node/refactor_attach_cppgc_heap_on_v8_isolate_creation.patch index 15b57e690506..1dce7f1de16f 100644 --- a/patches/node/refactor_attach_cppgc_heap_on_v8_isolate_creation.patch +++ b/patches/node/refactor_attach_cppgc_heap_on_v8_isolate_creation.patch @@ -165,10 +165,20 @@ index 4119ac1b002681d39711eac810ca2fcc2702ffc7..790347056cde949ffe6cf8498a7eca0c ExitCode NodeMainInstance::Run() { diff --git a/src/node_worker.cc b/src/node_worker.cc -index 1fc3774948dae3c0aae7d2aef563e18ecd4243a3..a610ee24ff18bddc3849aec3a43c2037b9ab5d53 100644 +index 1fc3774948dae3c0aae7d2aef563e18ecd4243a3..9d35cbf3dff538f38e8d5b8660d40c1fbaa56474 100644 --- a/src/node_worker.cc +++ b/src/node_worker.cc -@@ -230,13 +230,8 @@ class WorkerThreadData { +@@ -162,6 +162,9 @@ class WorkerThreadData { + SetIsolateCreateParamsForNode(¶ms); + w->UpdateResourceConstraints(¶ms.constraints); + params.array_buffer_allocator_shared = allocator; ++ params.cpp_heap = ++ v8::CppHeap::Create(w->platform_, v8::CppHeapCreateParams{{}}) ++ .release(); + Isolate* isolate = + NewIsolate(¶ms, &loop_, w->platform_, w->snapshot_data()); + if (isolate == nullptr) { +@@ -230,13 +233,8 @@ class WorkerThreadData { *static_cast(data) = true; }, &platform_finished);