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

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

View File

@@ -475,13 +475,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;
}