fix: make ExtendedClientDetails.refresh take a serializable parameter (#22759)

Author: mcollovati

flow-server/src/main/java/com/vaadin/flow/component/page/ExtendedClientDetails.java

@@ -18,7 +18,6 @@
 import java.io.Serializable;
 import java.util.Date;
 import java.util.TimeZone;
-import java.util.function.Consumer;
 import java.util.function.Function;
 
 import tools.jackson.databind.JsonNode;
@@ -37,7 +36,8 @@
  * {@link Page#getExtendedClientDetails()} and cached for the lifetime of the
  * UI. The fetch happens asynchronously, so the first call may return
  * {@code null} while the data is being retrieved. To update the cached values
- * with fresh data from the browser, use {@link #refresh(Consumer)}.
+ * with fresh data from the browser, use
+ * {@link #refresh(SerializableConsumer)}}.
  *
  * @author Vaadin Ltd
  * @since 2.0
@@ -459,7 +459,7 @@ public static ExtendedClientDetails fromJson(UI ui, JsonNode json) {
      *            a callback that will be invoked with the updated
      *            ExtendedClientDetails when the refresh is complete
      */
-    public void refresh(Consumer<ExtendedClientDetails> callback) {
+    public void refresh(SerializableConsumer<ExtendedClientDetails> callback) {
         final String js = "return Vaadin.Flow.getBrowserDetailsParameters();";
         final SerializableConsumer<JsonNode> resultHandler = json -> {
             ExtendedClientDetails details = fromJson(ui, json);

flow-server/src/main/java/com/vaadin/flow/component/page/Page.java

@@ -23,7 +23,6 @@
 import java.util.Arrays;
 import java.util.Objects;
 import java.util.UUID;
-import java.util.function.Consumer;
 
 import com.vaadin.flow.component.Direction;
 import com.vaadin.flow.component.UI;
@@ -488,11 +487,11 @@ public interface ExtendedClientDetailsReceiver extends Serializable {
      * If details are not yet available, this method returns a placeholder
      * instance with default values (dimensions set to -1). If you need to fetch
      * the actual values in such cases, use
-     * {@link ExtendedClientDetails#refresh(Consumer)} to explicitly retrieve
-     * updated values from the browser.
+     * {@link ExtendedClientDetails#refresh(SerializableConsumer)} to explicitly
+     * retrieve updated values from the browser.
      * <p>
      * To refresh the cached values with updated data from the browser at any
-     * time, use {@link ExtendedClientDetails#refresh(Consumer)}.
+     * time, use {@link ExtendedClientDetails#refresh(SerializableConsumer)}.
      *
      * @return the extended client details (never {@code null})
      */
@@ -508,7 +507,8 @@ public ExtendedClientDetails getExtendedClientDetails() {
      * @param receiver
      *            the callback to which the details are provided
      * @deprecated Use {@link #getExtendedClientDetails()} to get the cached
-     *             details, or {@link ExtendedClientDetails#refresh(Consumer)}
+     *             details, or
+     *             {@link ExtendedClientDetails#refresh(SerializableConsumer)}
      *             to refresh the cached values.
      */
     @Deprecated