Skip to content

Commit e4fdedd

Browse files
cjihrigjasnell
authored andcommitted
src: update v8::Object::GetPropertyNames() usage
Use the non-deprecated version of GetPropertyNames(). PR-URL: #23660 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com>
1 parent 9c5ec79 commit e4fdedd

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/node_contextify.cc

+6-1
Original file line numberDiff line numberDiff line change
@@ -508,7 +508,12 @@ void ContextifyContext::PropertyEnumeratorCallback(
508508
if (ctx->context_.IsEmpty())
509509
return;
510510

511-
args.GetReturnValue().Set(ctx->sandbox()->GetPropertyNames());
511+
Local<Array> properties;
512+
513+
if (!ctx->sandbox()->GetPropertyNames(ctx->context()).ToLocal(&properties))
514+
return;
515+
516+
args.GetReturnValue().Set(properties);
512517
}
513518

514519
// static

0 commit comments

Comments
 (0)