fix: values return from the ctx bridge with dynamic property support should themselves support dynamic properties (#28160)

Co-authored-by: Samuel Attard <sattard@slack-corp.com>
This commit is contained in:
Keeley Hammond
2021-03-12 17:39:06 -08:00
committed by GitHub
parent bb915813d7
commit 12a9970969
2 changed files with 45 additions and 2 deletions

View File

@@ -485,13 +485,15 @@ v8::MaybeLocal<v8::Object> CreateProxyForAPI(
v8::Local<v8::Value> setter_proxy;
if (!getter.IsEmpty()) {
if (!PassValueToOtherContext(source_context, destination_context,
getter, object_cache, false, 1)
getter, object_cache,
support_dynamic_properties, 1)
.ToLocal(&getter_proxy))
continue;
}
if (!setter.IsEmpty()) {
if (!PassValueToOtherContext(source_context, destination_context,
setter, object_cache, false, 1)
setter, object_cache,
support_dynamic_properties, 1)
.ToLocal(&setter_proxy))
continue;
}