Simplify String.repeat shim

Author: lukechilds

internal/assert/assertion_error.js

@@ -18,32 +18,10 @@ function endsWith(str, search, this_len) {
 
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat
 function repeat(str, count) {
-  if (str == null)
-    throw new TypeError('can\'t convert ' + this + ' to object');
-
-  str = '' + str;
-  // To convert string to integer.
-  count = +count;
-  // Check NaN
-  if (count != count)
-    count = 0;
-
-  if (count < 0)
-    throw new RangeError('repeat count must be non-negative');
-
-  if (count == Infinity)
-    throw new RangeError('repeat count must be less than infinity');
-
   count = Math.floor(count);
   if (str.length == 0 || count == 0)
     return '';
 
-  // Ensuring count is a 31-bit integer allows us to heavily optimize the
-  // main part. But anyway, most current (August 2014) browsers can't handle
-  // strings 1 << 28 chars or longer, so:
-  if (str.length * count >= 1 << 28)
-    throw new RangeError('repeat count must not overflow maximum string size');
-
   var maxCount = str.length * count;
   count = Math.floor(Math.log(count) / Math.log(2));
   while (count) {

test/common/index.js

@@ -10,32 +10,10 @@ const objectEntries = Object.entries
 
 // https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat
 function repeat(str, count) {
-  if (str == null)
-    throw new TypeError('can\'t convert ' + this + ' to object');
-
-  str = '' + str;
-  // To convert string to integer.
-  count = +count;
-  // Check NaN
-  if (count != count)
-    count = 0;
-
-  if (count < 0)
-    throw new RangeError('repeat count must be non-negative');
-
-  if (count == Infinity)
-    throw new RangeError('repeat count must be less than infinity');
-
   count = Math.floor(count);
   if (str.length == 0 || count == 0)
     return '';
 
-  // Ensuring count is a 31-bit integer allows us to heavily optimize the
-  // main part. But anyway, most current (August 2014) browsers can't handle
-  // strings 1 << 28 chars or longer, so:
-  if (str.length * count >= 1 << 28)
-    throw new RangeError('repeat count must not overflow maximum string size');
-
   var maxCount = str.length * count;
   count = Math.floor(Math.log(count) / Math.log(2));
   while (count) {