Revert "When objects are merged together, the target prototype can be…

… polluted. (chartjs#7918)"

This reverts commit dff7140.

src/helpers/helpers.core.js

@@ -1,9 +1,5 @@
 'use strict';
 
-function isValidKey(key) {
-	return ['__proto__', 'prototype', 'constructor'].indexOf(key) === -1;
-}
-
 /**
  * @namespace Chart.helpers
  */
@@ -200,12 +196,6 @@ var helpers = {
 	 * @private
 	 */
 	_merger: function(key, target, source, options) {
-		if (!isValidKey(key)) {
-			// We want to ensure we do not copy prototypes over
-			// as this can pollute global namespaces
-			return;
-		}
-
 		var tval = target[key];
 		var sval = source[key];
 
@@ -221,12 +211,6 @@ var helpers = {
 	 * @private
 	 */
 	_mergerIf: function(key, target, source) {
-		if (!isValidKey(key)) {
-			// We want to ensure we do not copy prototypes over
-			// as this can pollute global namespaces
-			return;
-		}
-
 		var tval = target[key];
 		var sval = source[key];
 

test/specs/helpers.core.tests.js

@@ -304,11 +304,6 @@ describe('Chart.helpers.core', function() {
 	});
 
 	describe('merge', function() {
-		it('should not allow prototype pollution', function() {
-			var test = helpers.merge({}, JSON.parse('{"__proto__":{"polluted": true}}'));
-			expect(test.prototype).toBeUndefined();
-			expect(Object.prototype.polluted).toBeUndefined();
-		});
 		it('should update target and return it', function() {
 			var target = {a: 1};
 			var result = helpers.merge(target, {a: 2, b: 'foo'});