From c5630e2699490e426d10fe712f0c19f79c341e88 Mon Sep 17 00:00:00 2001
From: Daeyeon Jeong <daeyeon.dev@gmail.com>
Date: Tue, 22 Nov 2022 16:14:59 +0900
Subject: [PATCH] src,node-api: update `napi_is_detached_arraybuffer`

This gets `napi_is_detached_arraybuffer(..)` to use the new API,
`ArrayBuffer::WasDetached`, that looks more straightforward.

Signed-off-by: Daeyeon Jeong <daeyeon.dev@gmail.com>
PR-URL: https://github.com/nodejs/node/pull/45538
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Michael Dawson <midawson@redhat.com>
---
 src/js_native_api_v8.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/js_native_api_v8.cc b/src/js_native_api_v8.cc
index 9000175d9d869a..abc9e56d6b0bbb 100644
--- a/src/js_native_api_v8.cc
+++ b/src/js_native_api_v8.cc
@@ -3254,7 +3254,7 @@ napi_status NAPI_CDECL napi_is_detached_arraybuffer(napi_env env,
   v8::Local<v8::Value> value = v8impl::V8LocalValueFromJsValue(arraybuffer);
 
   *result =
-      value->IsArrayBuffer() && value.As<v8::ArrayBuffer>()->Data() == nullptr;
+      value->IsArrayBuffer() && value.As<v8::ArrayBuffer>()->WasDetached();
 
   return napi_clear_last_error(env);
 }