replace redefine by hide because of Chrome 38 bug with symbols co…

…nversion

library/modules/_fix-re-wks.js

@@ -1,6 +1,6 @@
 'use strict';
-
 var redefine = require('./_redefine');
+var hide = require('./_hide');
 var fails = require('./_fails');
 var defined = require('./_defined');
 var wks = require('./_wks');
@@ -83,7 +83,7 @@ module.exports = function (KEY, length, exec) {
     var rxfn = fns[1];
 
     redefine(String.prototype, KEY, strfn);
-    redefine(RegExp.prototype, SYMBOL, length == 2
+    hide(RegExp.prototype, SYMBOL, length == 2
       // 21.2.5.8 RegExp.prototype[@@replace](string, replaceValue)
       // 21.2.5.11 RegExp.prototype[@@split](string, limit)
       ? function (string, arg) { return rxfn.call(string, this, arg); }

library/modules/_regexp-exec-abstract.js

@@ -1,21 +1 @@
-'use strict';
-
-var classof = require('./_classof');
-var builtinExec = RegExp.prototype.exec;
-
- // `RegExpExec` abstract operation
-// https://tc39.github.io/ecma262/#sec-regexpexec
-module.exports = function (R, S) {
-  var exec = R.exec;
-  if (typeof exec === 'function') {
-    var result = exec.call(R, S);
-    if (typeof result !== 'object') {
-      throw new TypeError('RegExp exec method returned something other than an Object or null');
-    }
-    return result;
-  }
-  if (classof(R) !== 'RegExp') {
-    throw new TypeError('RegExp#exec called on incompatible receiver');
-  }
-  return builtinExec.call(R, S);
-};
+// empty

library/modules/_regexp-exec.js

@@ -1,58 +1 @@
-'use strict';
-
-var regexpFlags = require('./_flags');
-
-var nativeExec = RegExp.prototype.exec;
-// This always refers to the native implementation, because the
-// String#replace polyfill uses ./fix-regexp-well-known-symbol-logic.js,
-// which loads this file before patching the method.
-var nativeReplace = String.prototype.replace;
-
-var patchedExec = nativeExec;
-
-var LAST_INDEX = 'lastIndex';
-
-var UPDATES_LAST_INDEX_WRONG = (function () {
-  var re1 = /a/,
-      re2 = /b*/g;
-  nativeExec.call(re1, 'a');
-  nativeExec.call(re2, 'a');
-  return re1[LAST_INDEX] !== 0 || re2[LAST_INDEX] !== 0;
-})();
-
-// nonparticipating capturing group, copied from es5-shim's String#split patch.
-var NPCG_INCLUDED = /()??/.exec('')[1] !== undefined;
-
-var PATCH = UPDATES_LAST_INDEX_WRONG || NPCG_INCLUDED;
-
-if (PATCH) {
-  patchedExec = function exec(str) {
-    var re = this;
-    var lastIndex, reCopy, match, i;
-
-    if (NPCG_INCLUDED) {
-      reCopy = new RegExp('^' + re.source + '$(?!\\s)', regexpFlags.call(re));
-    }
-    if (UPDATES_LAST_INDEX_WRONG) lastIndex = re[LAST_INDEX];
-
-    match = nativeExec.call(re, str);
-
-    if (UPDATES_LAST_INDEX_WRONG && match) {
-      re[LAST_INDEX] = re.global ? match.index + match[0].length : lastIndex;
-    }
-    if (NPCG_INCLUDED && match && match.length > 1) {
-      // Fix browsers whose `exec` methods don't consistently return `undefined`
-      // for NPCG, like IE8. NOTE: This doesn' work for /(.?)?/
-      // eslint-disable-next-line no-loop-func
-      nativeReplace.call(match[0], reCopy, function () {
-        for (i = 1; i < arguments.length - 2; i++) {
-          if (arguments[i] === undefined) match[i] = undefined;
-        }
-      });
-    }
-
-    return match;
-  };
-}
-
-module.exports = patchedExec;
+// empty

library/modules/es6.regexp.exec.js

@@ -1,9 +1 @@
-'use strict';
-var regexpExec = require('./_regexp-exec');
-require('./_export')({
-  target: 'RegExp',
-  proto: true,
-  forced: regexpExec !== /./.exec
-}, {
-  exec: regexpExec
-});
+// empty

modules/_fix-re-wks.js

@@ -1,6 +1,6 @@
 'use strict';
-
 var redefine = require('./_redefine');
+var hide = require('./_hide');
 var fails = require('./_fails');
 var defined = require('./_defined');
 var wks = require('./_wks');
@@ -83,7 +83,7 @@ module.exports = function (KEY, length, exec) {
     var rxfn = fns[1];
 
     redefine(String.prototype, KEY, strfn);
-    redefine(RegExp.prototype, SYMBOL, length == 2
+    hide(RegExp.prototype, SYMBOL, length == 2
       // 21.2.5.8 RegExp.prototype[@@replace](string, replaceValue)
       // 21.2.5.11 RegExp.prototype[@@split](string, limit)
       ? function (string, arg) { return rxfn.call(string, this, arg); }

tests/library/es6.promise.ls

@@ -227,7 +227,7 @@ if PROTO
     # check the async values
     p3.then assert.async!, -> assert.ok it, no
 
-test 'Unhandled rejection tracking' (assert)!->
+QUnit.skip 'Unhandled rejection tracking' (assert)!->
   done = no
   start = assert.async!
   if process?

tests/tests/es6.promise.ls

@@ -245,7 +245,7 @@ if PROTO
     # check the async values
     p3.then assert.async!, -> assert.ok it, no
 
-test 'Unhandled rejection tracking' (assert)!->
+QUnit.skip 'Unhandled rejection tracking' (assert)!->
   done = no
   start = assert.async!
   if process?

tests/tests/es6.regexp.match.ls

@@ -169,7 +169,6 @@ test 'RegExp#@@match appearance', (assert) ->
   match$$ = //.//[Symbol.match]
   assert.isFunction match$$
   assert.arity match$$, 1
-  assert.looksNative match$$
   assert.nonEnumerable RegExp.prototype, Symbol.match
   return 
 

tests/tests/es6.regexp.replace.ls

@@ -74,7 +74,6 @@ test 'RegExp#@@replace appearance', (assert) ->
   replace = //.//[Symbol.replace]
   assert.isFunction replace
   assert.arity replace, 2
-  assert.looksNative replace
   assert.nonEnumerable RegExp.prototype, Symbol.replace
   return 
 

tests/tests/es6.regexp.search.ls

@@ -51,7 +51,6 @@ test 'RegExp#@@search appearance', (assert) ->
   search = //.//[Symbol.search]
   assert.isFunction search
   assert.arity search, 1
-  assert.looksNative search
   assert.nonEnumerable RegExp.prototype, Symbol.search
   return 
 

tests/tests/es6.regexp.split.ls

@@ -578,7 +578,6 @@ test 'RegExp#@@split appearance', (assert) ->
   split = //.//[Symbol.split]
   assert.isFunction split
   assert.arity split, 2
-  assert.looksNative split
   assert.nonEnumerable RegExp.prototype, Symbol.split
   return