Fix Reflect.defineProperty on edge v0.12

paulmillr · Sep 6, 2015 · e8d7e65 · e8d7e65
1 parent 84ef88c
commit e8d7e65
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/es6-shim.js b/es6-shim.js
@@ -3184,9 +3184,11 @@
     }
   }
   if (globals.Reflect.defineProperty) {
-    if (valueOrFalseIfThrows(function () {
-      globals.Reflect.defineProperty(1, 'test', { value: 1 });
-      return true;
+    if (!valueOrFalseIfThrows(function () {
+      var basic = !globals.Reflect.defineProperty(1, 'test', { value: 1 });
+      // "extensible" fails on Edge 0.12
+      var extensible = typeof Object.preventExtensions !== 'function' || !globals.Reflect.defineProperty(Object.preventExtensions({}), 'test', {});
+      return basic && extensible;
     })) {
       overrideNative(globals.Reflect, 'defineProperty', ReflectShims.defineProperty);
     }