Fixes #4904 - WebsocketClient creates more connections than needed.

Updates after review.

Signed-off-by: Simone Bordet <[email protected]>

jetty-client/src/main/java/org/eclipse/jetty/client/HttpDestination.java

@@ -465,16 +465,14 @@ public boolean remove(Connection connection)
         {
             tryRemoveIdleDestination();
         }
-        else
+        else if (removed)
         {
-            // We need to execute queued requests
-            // even if this connection was removed.
+            // Process queued requests that may be waiting.
             // We may create a connection that is not
             // needed, but it will eventually idle timeout.
-            if (removed)
-                process(true);
+            process(true);
         }
-        return connectionPool.remove(connection);
+        return removed;
     }
 
     public void close(Connection connection)