nodejs
diff --git a/‎src/js_native_api_v8.cc‎
Lines changed: 1 addition & 1 deletion b/‎src/js_native_api_v8.cc‎
Lines changed: 1 addition & 1 deletion
diff --git a/‎src/node_blob.cc‎
Lines changed: 3 additions & 1 deletion b/‎src/node_blob.cc‎
Lines changed: 3 additions & 1 deletion
diff --git a/‎src/node_buffer.cc‎
Lines changed: 3 additions & 3 deletions b/‎src/node_buffer.cc‎
Lines changed: 3 additions & 3 deletions
diff --git a/‎src/node_messaging.cc‎
Lines changed: 1 addition & 1 deletion b/‎src/node_messaging.cc‎
Lines changed: 1 addition & 1 deletion
@@ -3239,7 +3239,7 @@ napi_status NAPI_CDECL napi_detach_arraybuffer(napi_env env,
   RETURN_STATUS_IF_FALSE(
       env, it->IsDetachable(), napi_detachable_arraybuffer_expected);
 
-  it->Detach();
+  it->Detach(v8::Local<v8::Value>()).Check();
 
   return napi_clear_last_error(env);
 }
 
@@ -107,7 +107,9 @@ void Blob::New(const FunctionCallbackInfo<Value>& args) {
       CHECK_EQ(view->ByteOffset(), 0);
       std::shared_ptr<BackingStore> store = view->Buffer()->GetBackingStore();
       size_t byte_length = view->ByteLength();
-      view->Buffer()->Detach();  // The Blob will own the backing store now.
+      view->Buffer()
+          ->Detach(Local<Value>())
+          .Check();  // The Blob will own the backing store now.
       entries.emplace_back(BlobEntry{std::move(store), byte_length, 0});
       len += byte_length;
     } else {
 
@@ -125,7 +125,7 @@ Local<ArrayBuffer> CallbackInfo::CreateTrackedArrayBuffer(
   // V8 simply ignores the BackingStore deleter callback if data == nullptr,
   // but our API contract requires it being called.
   if (data == nullptr) {
-    ab->Detach();
+    ab->Detach(Local<Value>()).Check();
     self->OnBackingStoreFree();  // This calls `callback` asynchronously.
   } else {
     // Store the ArrayBuffer so that we can detach it later.
@@ -156,7 +156,7 @@ void CallbackInfo::CleanupHook(void* data) {
     HandleScope handle_scope(self->env_->isolate());
     Local<ArrayBuffer> ab = self->persistent_.Get(self->env_->isolate());
     if (!ab.IsEmpty() && ab->IsDetachable()) {
-      ab->Detach();
+      ab->Detach(Local<Value>()).Check();
       self->persistent_.Reset();
     }
   }
@@ -1256,7 +1256,7 @@ void DetachArrayBuffer(const FunctionCallbackInfo<Value>& args) {
     Local<ArrayBuffer> buf = args[0].As<ArrayBuffer>();
     if (buf->IsDetachable()) {
       std::shared_ptr<BackingStore> store = buf->GetBackingStore();
-      buf->Detach();
+      buf->Detach(Local<Value>()).Check();
       args.GetReturnValue().Set(ArrayBuffer::New(env->isolate(), store));
     }
   }
 
@@ -516,7 +516,7 @@ Maybe<bool> Message::Serialize(Environment* env,
   for (Local<ArrayBuffer> ab : array_buffers) {
     // If serialization succeeded, we render it inaccessible in this Isolate.
     std::shared_ptr<BackingStore> backing_store = ab->GetBackingStore();
-    ab->Detach();
+    ab->Detach(Local<Value>()).Check();
 
     array_buffers_.emplace_back(std::move(backing_store));
   }
Original file line number	Diff line number	Diff line change
@@ -3239,7 +3239,7 @@ napi_status NAPI_CDECL napi_detach_arraybuffer(napi_env env,
3239	3239	RETURN_STATUS_IF_FALSE(
3240	3240	env, it->IsDetachable(), napi_detachable_arraybuffer_expected);
3241	3241
3242		- it->Detach();
	3242	+ it->Detach(v8::Local<v8::Value>()).Check();
3243	3243
3244	3244	return napi_clear_last_error(env);
3245	3245	}
Original file line number	Diff line number	Diff line change
@@ -516,7 +516,7 @@ Maybe<bool> Message::Serialize(Environment* env,
516	516	for (Local<ArrayBuffer> ab : array_buffers) {
517	517	// If serialization succeeded, we render it inaccessible in this Isolate.
518	518	std::shared_ptr<BackingStore> backing_store = ab->GetBackingStore();
519		- ab->Detach();
	519	+ ab->Detach(Local<Value>()).Check();
520	520
521	521	array_buffers_.emplace_back(std::move(backing_store));
522	522	}