Fix hasObjectSpliceBug implementations of _#pop, _#shift, and _#splice.

Former-commit-id: 91a3bc259c85bd269c3d895b66204bdc4d158827

lodash.js

@@ -4399,16 +4399,18 @@
   // avoid array-like object bugs with `Array#shift` and `Array#splice`
   // in Firefox < 10 and IE < 9
   if (hasObjectSpliceBug) {
-    each(['shift', 'splice'], function(methodName) {
+    each(['pop', 'shift', 'splice'], function(methodName) {
-      var func = lodash.prototype[methodName];
+      var func = arrayRef[methodName],
+          isSplice = methodName == 'splice';
+
       lodash.prototype[methodName] = function() {
         var value = this.__wrapped__,
-            result = func.apply(this, arguments);
+            result = func.apply(value, arguments);
 
         if (value.length === 0) {
           delete value[0];
         }
-        return result;
+        return isSplice ? new lodash(result) : result;
       };
     });
   }

test/underscore.html

@@ -34,8 +34,11 @@
 		</script>
 		<script>
 			(function() {
-				var push = Array.prototype.push,
+				var arrayProto = Array.prototype,
-				    slice = Array.prototype.slice;
+				    concat = arrayProto.concat,
+				    pop = arrayProto.pop,
+				    push = arrayProto.push,
+				    slice = arrayProto.slice;
 
 				if (_.chain) {
 					return;
@@ -70,19 +73,19 @@
 					return this;
 				};
 
-				_.prototype.concat = _.wrap(_.prototype.concat, function(func) {
+				_.prototype.concat = function() {
-					var result = func.apply(this, slice.call(arguments, 1));
+					var result = concat.apply(this.__wrapped__, arguments);
 					if (this.__chain__) {
 						result = new _(result);
 						result.__chain__ = true;
 					}
 					return result;
-				});
+				};
 
-				_.prototype.pop = _.wrap(_.prototype.pop, function(func) {
+				_.prototype.pop = function() {
-					func.apply(this, slice.call(arguments, 1));
+					pop.apply(this.__wrapped__, arguments);
 					return this;
-				});
+				};
 			}());
 		</script>
 		<script src="../vendor/underscore/test/collections.js"></script>