src: do not rely on the internal field being default to undefined

It's an undocumented V8 behavior that is subject to change. Instead
just check if the internal field is set to a promise. There is also
no need to check IsEmpty() since the object is guaranteed to be
constructed by the FileHandle constructor with enough internal
fields.

PR-URL: #49413
Refs: https://chromium-review.googlesource.com/c/v8/v8/+/4707972/comment/be9285cc_a49aad88/
Reviewed-By: Yagiz Nizipli <[email protected]>

Author: joyeecheung

src/node_file.cc

@@ -457,8 +457,7 @@ MaybeLocal<Promise> FileHandle::ClosePromise() {
 
   Local<Value> close_resolver =
       object()->GetInternalField(FileHandle::kClosingPromiseSlot).As<Value>();
-  if (!close_resolver.IsEmpty() && !close_resolver->IsUndefined()) {
-    CHECK(close_resolver->IsPromise());
+  if (close_resolver->IsPromise()) {
     return close_resolver.As<Promise>();
   }