Fixes zloirock#173. Symbol()'s call to Object.defineProperty() access…

…or descriptor should set a default 'get' property when adding a 'set' property. Prevents crash in old Qt Script engine (WebKit).
bendiy · Feb 25, 2016 · 5c7fef8 · 5c7fef8
1 parent 7bc6210
commit 5c7fef8
Show file tree

Hide file tree

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

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