Skip to content

Commit f5cadf6

Browse files
cjihrigMylesBorins
authored andcommitted
src: update v8::Object::GetPropertyNames() usage
Use the non-deprecated version of GetPropertyNames(). PR-URL: #23660 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Gus Caplan <[email protected]> Reviewed-By: Sakthipriyan Vairamani <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent 3e4032a commit f5cadf6

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/node_contextify.cc

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -504,7 +504,12 @@ void ContextifyContext::PropertyEnumeratorCallback(
504504
if (ctx->context_.IsEmpty())
505505
return;
506506

507-
args.GetReturnValue().Set(ctx->sandbox()->GetPropertyNames());
507+
Local<Array> properties;
508+
509+
if (!ctx->sandbox()->GetPropertyNames(ctx->context()).ToLocal(&properties))
510+
return;
511+
512+
args.GetReturnValue().Set(properties);
508513
}
509514

510515
// static

0 commit comments

Comments
 (0)