fix! stop enabling O.p.constructor

endojs · Feb 5, 2021 · a5b165b · a5b165b
1 parent bf06c3a
commit a5b165b
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 7 deletions.
diff --git a/packages/ses/src/enablements.js b/packages/ses/src/enablements.js
@@ -58,7 +58,6 @@
 
 export const moderateEnablements = {
   '%ObjectPrototype%': {
-    constructor: true, // set by "acorn" v7.
     hasOwnProperty: true, // set by "vega-util".
     toString: true,
     valueOf: true,

diff --git a/packages/ses/test/test-enable-property-overrides.js b/packages/ses/test/test-enable-property-overrides.js
@@ -91,12 +91,7 @@ test('enablePropertyOverrides - on', t => {
 
   harden(intrinsics);
 
-  testOverriding(t, 'Object', {}, [
-    'constructor',
-    'hasOwnProperty',
-    'toString',
-    'valueOf',
-  ]);
+  testOverriding(t, 'Object', {}, ['hasOwnProperty', 'toString', 'valueOf']);
   // We allow 'length' *not* because it is in enablements; it is not;
   // but because each array instance has its own.
   testOverriding(t, 'Array', [], ['toString', 'length']);