Fixes: zloirock#173 Change 'Symbol()' wrap to use a 'data descriptor'…

… for 'Object.defineProperty()' instead of an 'accessor descriptor'. Fixes RegExp.exec() creash in Qt Script.
bendiy · Feb 26, 2016 · 1f4c759 · 1f4c759
1 parent 7bc6210
commit 1f4c759
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/library/modules/es6.symbol.js b/library/modules/es6.symbol.js
@@ -54,10 +54,8 @@ var wrap = function(tag){
   sym._k = tag;
   DESCRIPTORS && setter && setSymbolDesc(ObjectProto, tag, {
     configurable: true,
-    set: function(value){
-      if(has(this, HIDDEN) && has(this[HIDDEN], tag))this[HIDDEN][tag] = false;
-      setSymbolDesc(this, tag, createDesc(1, value));
-    }
+    enumerable: false,
+    writable: true
   });
   return sym;
 };

diff --git a/modules/es6.symbol.js b/modules/es6.symbol.js
@@ -54,10 +54,8 @@ var wrap = function(tag){
   sym._k = tag;
   DESCRIPTORS && setter && setSymbolDesc(ObjectProto, tag, {
     configurable: true,
-    set: function(value){
-      if(has(this, HIDDEN) && has(this[HIDDEN], tag))this[HIDDEN][tag] = false;
-      setSymbolDesc(this, tag, createDesc(1, value));
-    }
+    enumerable: false,
+    writable: true
   });
   return sym;
 };