fixup! fixup! timers: use V8 fast API calls

Author: joyeecheung

lib/internal/timers.js

@@ -303,13 +303,17 @@ class ImmediateList {
 const immediateQueue = new ImmediateList();
 
 function incRefCount() {
-  if (timeoutInfo[0]++ === 0)
+  if (timeoutInfo[0]++ === 0) {
+    // We need to use the binding as the receiver for fast API calls.
     binding.toggleTimerRef(true);
+  }
 }
 
 function decRefCount() {
-  if (--timeoutInfo[0] === 0)
+  if (--timeoutInfo[0] === 0) {
+    // We need to use the binding as the receiver for fast API calls.
     binding.toggleTimerRef(false);
+  }
 }
 
 // Schedule or re-schedule a timer.
@@ -353,6 +357,7 @@ function insertGuarded(item, refed, start) {
   item[kRefed] = refed;
 }
 
+// We need to use the binding as the receiver for fast API calls.
 function insert(item, msecs, start = binding.getLibuvNow()) {
   // Truncate so that accuracy of sub-millisecond timers is not assumed.
   msecs = MathTrunc(msecs);
@@ -367,6 +372,7 @@ function insert(item, msecs, start = binding.getLibuvNow()) {
     timerListQueue.insert(list);
 
     if (nextExpiry > expiry) {
+      // We need to use the binding as the receiver for fast API calls.
       binding.scheduleTimer(msecs);
       nextExpiry = expiry;
     }
@@ -556,8 +562,10 @@ function getTimerCallbacks(runNextTicks) {
       emitBefore(asyncId, timer[trigger_async_id_symbol], timer);
 
       let start;
-      if (timer._repeat)
+      if (timer._repeat) {
+        // We need to use the binding as the receiver for fast API calls.
         start = binding.getLibuvNow();
+      }
 
       try {
         const args = timer._timerArgs;
@@ -624,17 +632,22 @@ class Immediate {
   ref() {
     if (this[kRefed] === false) {
       this[kRefed] = true;
-      if (immediateInfo[kRefCount]++ === 0)
+
+      if (immediateInfo[kRefCount]++ === 0) {
+       // We need to use the binding as the receiver for fast API calls.
         binding.toggleImmediateRef(true);
+      }
     }
     return this;
   }
 
   unref() {
     if (this[kRefed] === true) {
       this[kRefed] = false;
-      if (--immediateInfo[kRefCount] === 0)
+      if (--immediateInfo[kRefCount] === 0) {
+        // We need to use the binding as the receiver for fast API calls.
         binding.toggleImmediateRef(false);
+      }
     }
     return this;
   }

lib/timers.js

@@ -323,8 +323,10 @@ function clearImmediate(immediate) {
   immediateInfo[kCount]--;
   immediate._destroyed = true;
 
-  if (immediate[kRefed] && --immediateInfo[kRefCount] === 0)
+  if (immediate[kRefed] && --immediateInfo[kRefCount] === 0) {
+    // We need to use the binding as the receiver for fast API calls.
     binding.toggleImmediateRef(false);
+  }
   immediate[kRefed] = null;
 
   if (destroyHooksExist() && immediate[async_id_symbol] !== undefined) {

src/timers.h

@@ -34,7 +34,7 @@ class BindingData : public SnapshotableObject {
   static void SlowScheduleTimer(
       const v8::FunctionCallbackInfo<v8::Value>& args);
   static void FastScheduleTimer(v8::Local<v8::Object> receiver,
-                                 int64_t duration);
+                                int64_t duration);
   static void ScheduleTimerImpl(BindingData* data, int64_t duration);
 
   static void SlowToggleTimerRef(

test/parallel/test-timers-now.js

@@ -7,4 +7,5 @@ const { internalBinding } = require('internal/test/binding');
 const binding = internalBinding('timers');
 
 // Return value of getLibuvNow() should easily fit in a SMI after start-up.
+// We need to use the binding as the receiver for fast API calls.
 assert(binding.getLibuvNow() < 0x3ffffff);

test/parallel/test-timers-ordering.js

@@ -39,6 +39,7 @@ function f(i) {
     last_i = i;
 
     // Check that this iteration is fired at least 1ms later than the previous
+    // We need to use the binding as the receiver for fast API calls.
     const now = binding.getLibuvNow();
     assert(now >= last_ts + 1,
            `current ts ${now} < prev ts ${last_ts} + 1`);