timers: avoid generating holey internal arrays

PR-URL: #54771
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>

lib/timers.js

@@ -22,6 +22,7 @@
 'use strict';
 
 const {
+ ArrayPrototypePush,
  MathTrunc,
  ObjectDefineProperties,
  ObjectDefineProperty,
@@ -154,7 +155,7 @@ function setTimeout(callback, after, arg1, arg2, arg3) {
  args = [arg1, arg2, arg3];
  for (i = 5; i < arguments.length; i++) {
  // Extend array dynamically, makes .apply run much faster in v6.0.0
- args[i - 2] = arguments[i];
+ ArrayPrototypePush(args, arguments[i]);
  }
  break;
  }
@@ -224,7 +225,7 @@ function setInterval(callback, repeat, arg1, arg2, arg3) {
  args = [arg1, arg2, arg3];
  for (i = 5; i < arguments.length; i++) {
  // Extend array dynamically, makes .apply run much faster in v6.0.0
- args[i - 2] = arguments[i];
+ ArrayPrototypePush(args, arguments[i]);
  }
  break;
  }
@@ -296,7 +297,7 @@ function setImmediate(callback, arg1, arg2, arg3) {
  args = [arg1, arg2, arg3];
  for (i = 4; i < arguments.length; i++) {
  // Extend array dynamically, makes .apply run much faster in v6.0.0
- args[i - 1] = arguments[i];
+ ArrayPrototypePush(args, arguments[i]);
  }
  break;
  }