From 0be9e2a33ffcf9a8f6c30ccc1623566838313d62 Mon Sep 17 00:00:00 2001 From: zodac Date: Thu, 27 Mar 2025 22:59:19 +1300 Subject: [PATCH] [java] Removing old stream collectors required by Java 8, using Java 10 implementations --- .../selenium/PersistentCapabilities.java | 18 ++---------------- 1 file changed, 2 insertions(+), 16 deletions(-) diff --git a/java/src/org/openqa/selenium/PersistentCapabilities.java b/java/src/org/openqa/selenium/PersistentCapabilities.java index d42e08229b6a5..8da7482bf6e4b 100644 --- a/java/src/org/openqa/selenium/PersistentCapabilities.java +++ b/java/src/org/openqa/selenium/PersistentCapabilities.java @@ -17,11 +17,9 @@ package org.openqa.selenium; -import java.util.Collections; import java.util.Map; import java.util.Set; import java.util.function.Function; -import java.util.stream.Collector; import java.util.stream.Collectors; import java.util.stream.Stream; import org.openqa.selenium.internal.Require; @@ -58,7 +56,7 @@ public PersistentCapabilities setCapability(String name, Object value) { @Override public Map asMap() { return getCapabilityNames().stream() - .collect(toUnmodifiableMap(Function.identity(), this::getCapability)); + .collect(Collectors.toUnmodifiableMap(Function.identity(), this::getCapability)); } @Override @@ -81,19 +79,7 @@ public Capabilities merge(Capabilities other) { public Set getCapabilityNames() { return Stream.concat( caps.getCapabilityNames().stream(), overrides.getCapabilityNames().stream()) - .collect(toUnmodifiableSet()); - } - - // Needed, since we're dependent on Java 8 as a minimum version - private Collector> toUnmodifiableMap( - Function keyMapper, Function valueMapper) { - return Collectors.collectingAndThen( - Collectors.toMap(keyMapper, valueMapper), Collections::unmodifiableMap); - } - - // Needed, since we're dependent on Java 8 as a minimum version - private Collector> toUnmodifiableSet() { - return Collectors.collectingAndThen(Collectors.toSet(), Collections::unmodifiableSet); + .collect(Collectors.toUnmodifiableSet()); } @Override