[experimental] add fix for Array#sort, #156

build/config.js

@@ -1,6 +1,7 @@
 module.exports = {
   list: [
     'es5',
+    'es5.array.sort',
     'es6.symbol',
     'es6.object.assign',
     'es6.object.is',
@@ -157,6 +158,7 @@ module.exports = {
     'core.string.unescape-html'
   ],
   experimental: [
+    'es5.array.sort',
     'es7.reflect.define-metadata',
     'es7.reflect.delete-metadata',
     'es7.reflect.get-metadata',

library/modules/es5.array.sort.js

@@ -0,0 +1,19 @@
+'use strict';
+var $export   = require('./_export')
+  , aFunction = require('./_a-function')
+  , fails     = require('./_fails')
+  , $sort     = [].sort
+  , test      = [1, 2, 3];
+
+$export($export.P + $export.F * (fails(function(){
+  // IE8-
+  test.sort(undefined);
+}) || !fails(function(){
+  // V8 bug
+  test.sort(null);
+})), 'Array', {
+  // 22.1.3.25 Array.prototype.sort(comparefn)
+  sort: function sort(comparefn){
+    return comparefn === undefined ? $sort.call(this) : $sort.call(this, aFunction(comparefn));
+  }
+});

modules/es5.array.sort.js

@@ -0,0 +1,19 @@
+'use strict';
+var $export   = require('./_export')
+  , aFunction = require('./_a-function')
+  , fails     = require('./_fails')
+  , $sort     = [].sort
+  , test      = [1, 2, 3];
+
+$export($export.P + $export.F * (fails(function(){
+  // IE8-
+  test.sort(undefined);
+}) || !fails(function(){
+  // V8 bug
+  test.sort(null);
+})), 'Array', {
+  // 22.1.3.25 Array.prototype.sort(comparefn)
+  sort: function sort(comparefn){
+    return comparefn === undefined ? $sort.call(this) : $sort.call(this, aFunction(comparefn));
+  }
+});

tests/experimental/es5.array.sort.ls

@@ -0,0 +1,16 @@
+{module, test} = QUnit
+module \ES5
+
+test 'Array#sort' (assert)!->
+  assert.isFunction Array::sort
+  assert.arity Array::sort, 1
+  assert.name Array::sort, \sort
+  assert.looksNative Array::sort
+  assert.looksNative Array::sort
+  assert.ok !!(try [1 2 3]sort void), 'works with undefined'
+  assert.throws (!-> [1 2 3]sort null), 'throws on null'
+  assert.throws (!-> [1 2 3]sort {}), 'throws on {}'
+  if STRICT
+    assert.throws (-> Array::sort.call null), TypeError, 'ToObject(this)'
+    assert.throws (-> Array::sort.call void), TypeError, 'ToObject(this)'
+  if NATIVE => assert.ok !!(try Array::sort.call {0: 1, 1: 3, 2: 2, length: -1}, -> throw 42), 'ToLength(? Get(obj, "length"))'