[bidi] Prefer returning immutable versions of values when serialising to JSON

Author: shs96c

java/src/org/openqa/selenium/bidi/browsingcontext/BoxClipRectangle.java

@@ -33,6 +33,6 @@ public BoxClipRectangle(double x, double y, double width, double height) {
   public Map<String, Object> toMap() {
     map.put("type", super.getType().toString());
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/browsingcontext/CaptureScreenshotParameters.java

@@ -61,6 +61,6 @@ public CaptureScreenshotParameters clipRectangle(ClipRectangle clipRectangle) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/browsingcontext/CreateContextParameters.java

@@ -46,6 +46,6 @@ public CreateContextParameters userContext(String userContext) {
 
   public Map<String, Object> toMap() {
     map.put("type", windowType.toString());
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/browsingcontext/ElementClipRectangle.java

@@ -36,6 +36,6 @@ public ElementClipRectangle(String sharedId, String handle) {
   public Map<String, Object> toMap() {
     map.put("type", super.getType().toString());
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/browsingcontext/LocateNodeParameters.java

@@ -71,6 +71,6 @@ public Map<String, Object> toMap() {
           map.put("startNodes", startNodesJson);
         });
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/browsingcontext/Locator.java

@@ -80,6 +80,6 @@ public static Locator xpath(String value) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/AddInterceptParameters.java

@@ -62,6 +62,6 @@ public Map<String, Object> toMap() {
     if (!urlPatterns.isEmpty()) {
       map.put("urlPatterns", urlPatterns);
     }
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/BytesValue.java

@@ -85,6 +85,6 @@ public Map<String, String> toMap() {
     map.put("type", type.toString());
     map.put("value", value);
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/ContinueRequestParameters.java

@@ -60,6 +60,6 @@ public ContinueRequestParameters url(http://github.com/String%20url) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/ContinueResponseParameters.java

@@ -68,6 +68,6 @@ public ContinueResponseParameters statusCode(int statusCode) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/Cookie.java

@@ -182,6 +182,6 @@ public Map<String, Object> toMap() {
 
     getExpiry().ifPresent(expiryValue -> map.put("expiry", expiryValue));
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/Header.java

@@ -66,6 +66,6 @@ public Map<String, Object> toMap() {
     map.put("name", this.name);
     map.put("value", this.value.toMap());
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/ProvideResponseParameters.java

@@ -59,6 +59,6 @@ public ProvideResponseParameters statusCode(int statusCode) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/SetCookieHeader.java

@@ -64,6 +64,6 @@ public SetCookieHeader expiry(long expiry) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/network/UrlPattern.java

@@ -50,6 +50,6 @@ public UrlPattern search(String search) {
 
   public Map<String, String> toMap() {
     map.put("type", "pattern");
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/script/CallFunctionParameters.java

@@ -57,6 +57,6 @@ public CallFunctionParameters userActivation(boolean userActivation) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/script/EvaluateParameters.java

@@ -46,6 +46,6 @@ public EvaluateParameters userActivation(boolean userActivation) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/BrowsingContextPartitionDescriptor.java

@@ -33,6 +33,6 @@ public Map<String, Object> toMap() {
     map.put("type", super.type.toString());
     map.put("context", this.context);
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/CookieFilter.java

@@ -71,6 +71,6 @@ public CookieFilter expiry(long expiry) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/DeleteCookiesParameters.java

@@ -43,6 +43,6 @@ public Map<String, Object> toMap() {
     cookieFilter.ifPresent(filter -> map.put("filter", cookieFilter));
     partitionDescriptor.ifPresent(descriptor -> map.put("partition", partitionDescriptor));
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/GetCookiesParameters.java

@@ -42,6 +42,6 @@ public Map<String, Object> toMap() {
     cookieFilter.ifPresent(filter -> map.put("filter", filter));
     partitionDescriptor.ifPresent(descriptor -> map.put("partition", descriptor));
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/PartialCookie.java

@@ -62,6 +62,6 @@ public PartialCookie expiry(long expiry) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/SetCookieParameters.java

@@ -38,6 +38,6 @@ public Map<String, Object> toMap() {
     map.put("cookie", cookie.toMap());
     partitionDescriptor.ifPresent(descriptor -> map.put("partition", descriptor));
 
-    return map;
+    return Map.copyOf(map);
   }
 }

java/src/org/openqa/selenium/bidi/storage/StorageKeyPartitionDescriptor.java

@@ -39,6 +39,6 @@ public StorageKeyPartitionDescriptor sourceOrigin(String sourceOrigin) {
   }
 
   public Map<String, Object> toMap() {
-    return map;
+    return Map.copyOf(map);
   }
 }