deps: V8: cherry-pick 25650aa994

Original commit message:

    [microtask] Early-exit empty PerformCheckpointInternal and OnCompleted

    Microtask checkpoints (MicrotaskQueue::PerformCheckpoint) are executed
    frequently at the end of every DOM event listener, microtask scope exit,
    timer callback, and mutation observer turn.

    In typical DOM interactions and event dispatch cycles, the microtask
    queue is frequently empty (size_ == 0), no microtask completion
    callbacks are registered, and no weak references were kept during the
    job (weak_refs_keep_during_job == undefined_value()).

    Previously, even when the queue was empty, PerformCheckpointInternal
    unconditionally allocated an artificial
    MicrotasksScope(kDoNotRunMicrotasks), called RunMicrotasks (which set
    running state flags, created suppress scopes, and rewound entered
    contexts), invoked OnCompleted, and called isolate->ClearKeptObjects()
    (performing an unconditional store to the weak_refs_keep_during_job root
    table slot).

    This CL fast-paths PerformCheckpointInternal when size() == 0,
    microtasks_completed_callbacks_ is empty, and weak_refs_keep_during_job
    is undefined. If WeakRef.prototype.deref() was invoked during the turn,
    weak_refs_keep_during_job holds an OrderedHashSet, so the early-exit is
    skipped and ClearKeptObjects() runs to clear kept objects as required by
    ECMAScript §26.1.

    Pinpoint Try Job on Apple Silicon M1 (150 iterations, Job
    158c2910690000):
    - Speedometer 3 Score: 31.42 -> 31.48 (+0.21%, p=0.0293, stat-sig win)
    - TodoMVC-JavaScript-ES6-Webpack: -0.87% (p=0.0001)
    - NewsSite-Nuxt: -0.40% (p=0.0001)
    - TodoMVC-Angular: -0.45% (p=0.0681)
    - TodoMVC-jQuery: -0.36% (p=0.6023)
    - Charts-chartjs: -0.21% (p=0.0005)
    - React-Stockcharts-SVG: -0.26% (p=0.0144)

    Bug: 553846289
    TAG=agy
    CONV=a3721850-14cb-425b-8d34-169f9934ed78

    Change-Id: Ic996d283b746a5684a55c9fbc56d8fcfb38ac299
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/8312487
    Commit-Queue: Anton Bikineev <bikineev@chromium.org>
    Auto-Submit: Anton Bikineev <bikineev@chromium.org>
    Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#109567}

Refs: v8/v8@25650aa
Signed-off-by: Nigro Simone <nigro.simone@gmail.com>

Author: nigrosimone

common.gypi

@@ -44,7 +44,7 @@
 
     # Reset this number to 0 on major V8 upgrades.
     # Increment by one for each non-official patch applied to deps/v8.
-    'v8_embedder_string': '-node.22',
+    'v8_embedder_string': '-node.23',
 
     ##### V8 defaults for Node.js #####
 

deps/v8/src/execution/microtask-queue.cc

@@ -19,6 +19,7 @@
 #include "src/execution/isolate.h"
 #include "src/handles/handle-scope-implementer-inl.h"
 #include "src/handles/handles-inl.h"
+#include "src/heap/heap-inl.h"
 #include "src/objects/microtask-inl.h"
 #include "src/objects/visitors.h"
 #include "src/roots/roots-inl.h"
@@ -165,6 +166,18 @@ void MicrotaskQueue::EnqueueMicrotask(Tagged<Microtask> microtask) {
 
 void MicrotaskQueue::PerformCheckpointInternal(v8::Isolate* v8_isolate) {
   DCHECK(ShouldPerfomCheckpoint());
+  DCHECK(!microtasks_completed_callbacks_cow_.has_value());
+  Isolate* isolate = reinterpret_cast<Isolate*>(v8_isolate);
+  // Fast path: Checkpoints occur frequently when exiting script or microtask
+  // scopes. If there are no microtasks to drain, no completion callbacks to
+  // notify, and no kept objects from FinalizationRegistry / WeakRefs to clear,
+  // we can bail out immediately and avoid artificial MicrotasksScope setup and
+  // RunMicrotasks overhead.
+  if (size() == 0 && microtasks_completed_callbacks_.empty() &&
+      isolate->heap()->weak_refs_keep_during_job() ==
+          ReadOnlyRoots(isolate).undefined_value()) [[likely]] {
+    return;
+  }
   std::optional<MicrotasksScope> microtasks_scope;
   if (microtasks_policy_ == v8::MicrotasksPolicy::kScoped) {
     // If we're using microtask scopes to schedule microtask execution, V8
@@ -175,7 +188,6 @@ void MicrotaskQueue::PerformCheckpointInternal(v8::Isolate* v8_isolate) {
     microtasks_scope.emplace(v8_isolate, this,
                              v8::MicrotasksScope::kDoNotRunMicrotasks);
   }
-  Isolate* isolate = reinterpret_cast<Isolate*>(v8_isolate);
   RunMicrotasks(isolate);
   isolate->ClearKeptObjects();
 }
@@ -347,12 +359,18 @@ void MicrotaskQueue::RemoveMicrotasksCompletedCallback(
 }
 
 void MicrotaskQueue::OnCompleted(Isolate* isolate) {
+  DCHECK_IMPLIES(microtasks_completed_callbacks_.empty(),
+                 !microtasks_completed_callbacks_cow_.has_value());
+  if (microtasks_completed_callbacks_.empty()) [[likely]] {
+    return;
+  }
+
   is_running_completed_callbacks_ = true;
   for (auto& callback : microtasks_completed_callbacks_) {
     callback.first(reinterpret_cast<v8::Isolate*>(isolate), callback.second);
   }
   is_running_completed_callbacks_ = false;
-  if (V8_UNLIKELY(microtasks_completed_callbacks_cow_.has_value())) {
+  if (microtasks_completed_callbacks_cow_.has_value()) [[unlikely]] {
     microtasks_completed_callbacks_ =
         std::move(microtasks_completed_callbacks_cow_.value());
     microtasks_completed_callbacks_cow_.reset();