Chromium Code Reviews| Index: src/array.js |
| diff --git a/src/array.js b/src/array.js |
| index 93378cfb00b1b14c0150e5b179c6de5f6a8852a2..0124b414bee21e6d7064b3565e3c6178644472b4 100644 |
| --- a/src/array.js |
| +++ b/src/array.js |
| @@ -12,6 +12,7 @@ var $arraySplice; |
| var $arrayUnshift; |
| var $innerArrayForEach; |
| var $innerArrayEvery; |
| +var $innerArrayReverse; |
| (function(global, shared, exports) { |
| @@ -564,12 +565,7 @@ function SparseReverse(array, len) { |
| } |
| -function ArrayReverse() { |
| - CHECK_OBJECT_COERCIBLE(this, "Array.prototype.reverse"); |
| - |
| - var array = TO_OBJECT_INLINE(this); |
| - var len = TO_UINT32(array.length); |
| - |
| +function InnerArrayReverse(array, len) { |
| if (UseSparseVariant(array, len, IS_ARRAY(array), len)) { |
|
arv (Not doing code reviews)
2015/05/19 15:18:02
This is always false for typed arrays. This can th
dehrenberg
2015/05/19 16:22:25
Done.
|
| %NormalizeElements(array); |
| SparseReverse(array, len); |
| @@ -600,6 +596,16 @@ function ArrayReverse() { |
| } |
| +function ArrayReverse() { |
| + CHECK_OBJECT_COERCIBLE(this, "Array.prototype.reverse"); |
| + |
| + var array = TO_OBJECT_INLINE(this); |
| + var len = TO_UINT32(array.length); |
| + |
| + return InnerArrayReverse(array, len); |
| +} |
| + |
| + |
| function ObservedArrayShift(len) { |
| var first = this[0]; |
| @@ -1609,5 +1615,6 @@ $arrayUnshift = ArrayUnshift; |
| $innerArrayForEach = InnerArrayForEach; |
| $innerArrayEvery = InnerArrayEvery; |
| +$innerArrayReverse = InnerArrayReverse; |
| }); |