wrapping apply and construct hooks instead of providing Reflect.* dir…

…ectly to avoid issue 1042435 in blick/chromium
salesforce · Jan 20, 2020 · 04ac821 · 04ac821
1 parent fb04f14
commit 04ac821
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/environment.ts b/src/environment.ts
@@ -75,8 +75,12 @@ export class SecureEnvironment implements MembraneBroker {
         // the proper errors without leaking instances into a sandbox
         const secureEnvFactory = secureGlobalThis.eval(`(${serializedSecureEnvSourceText})`);
         const rawHooks: MarshalHooks = {
-            apply,
-            construct,
+            apply(target: RawFunction, thisArgument: RawValue, argumentsList: ArrayLike<RawValue>): RawValue {
+                return apply(target, thisArgument, argumentsList);
+            },
+            construct(target: RawConstructor, argumentsList: ArrayLike<RawValue>, newTarget?: any): RawValue {
+                return construct(target, argumentsList, newTarget);
+            },
         };
         this.getSecureValue = secureEnvFactory(this, rawHooks);
         this.getRawValue = reverseProxyFactory(this);