[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(String url) {
   }
 
   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);
   }
 }