fix: Fixed high CPU usage due to pending JS commands (#22024) (#22035) (#22048)

Field "pendingJsInvocations" should be a Set instead of a List in order
to avoid O(n^2) operations. This greatly improves performance in
situations where many fields are updated constantly and the JS command
queue becomes very large.

Fixes #22024

Co-authored-by: Tomi Virtanen <tltv@vaadin.com>

(cherry picked from commit dfe8763)

Co-authored-by: daniel-pss-prosoft <54983789+daniel-pss-prosoft@users.noreply.github.com>

Author: mshabarov

flow-server/src/main/java/com/vaadin/flow/component/internal/UIInternals.java

@@ -16,6 +16,7 @@
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.IdentityHashMap;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -162,7 +163,7 @@ public List<Object> getParameters() {
      */
     private long lastHeartbeatTimestamp = System.currentTimeMillis();
 
-    private List<PendingJavaScriptInvocation> pendingJsInvocations = new ArrayList<>();
+    private Set<PendingJavaScriptInvocation> pendingJsInvocations = new LinkedHashSet<>();
 
     private final HashMap<StateNode, PendingJavaScriptInvocationDetachListener> pendingJsInvocationDetachListeners = new HashMap<>();
 
@@ -587,7 +588,7 @@ public List<PendingJavaScriptInvocation> dumpPendingJavaScriptInvocations() {
 
         pendingJsInvocations = getPendingJavaScriptInvocations()
                 .filter(invocation -> !invocation.getOwner().isVisible())
-                .collect(Collectors.toCollection(ArrayList::new));
+                .collect(Collectors.toCollection(LinkedHashSet::new));
         pendingJsInvocations
                 .forEach(this::registerDetachListenerForPendingInvocation);
         return readyToSend;
@@ -630,8 +631,7 @@ public void execute() {
 
         private void removePendingInvocation(
                 PendingJavaScriptInvocation invocation) {
-            UIInternals.this.pendingJsInvocations.removeIf(
-                    pendingInvocation -> pendingInvocation.equals(invocation));
+            UIInternals.this.pendingJsInvocations.remove(invocation);
             if (invocationList.isEmpty() && registration != null) {
                 registration.remove();
                 registration = null;