fix(sentry-java): Contexts belong on the Scope

sentry-core/src/main/java/io/sentry/core/Breadcrumb.java

@@ -1,7 +1,7 @@
 package io.sentry.core;
 
+import io.sentry.core.util.CollectionUtils;
 import java.util.Date;
-import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
@@ -227,20 +227,7 @@ Map<String, Object> getUnknown() {
       clone.data = null;
     }
 
-    final Map<String, Object> unknownRef = unknown;
-    if (unknownRef != null) {
-      final Map<String, Object> unknownClone = new HashMap<>();
-
-      for (Map.Entry<String, Object> item : unknownRef.entrySet()) {
-        if (item != null) {
-          unknownClone.put(item.getKey(), item.getValue()); // shallow copy
-        }
-      }
-
-      clone.unknown = unknownClone;
-    } else {
-      clone.unknown = null;
-    }
+    clone.unknown = CollectionUtils.shallowCopy(unknown);
 
     final SentryLevel levelRef = level;
     clone.level =

sentry-core/src/main/java/io/sentry/core/Scope.java

@@ -1,8 +1,8 @@
 package io.sentry.core;
 
+import io.sentry.core.protocol.Contexts;
 import io.sentry.core.protocol.User;
 import java.util.ArrayList;
-import java.util.HashMap;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -50,6 +50,9 @@ public final class Scope implements Cloneable {
   /** Session lock, Ops should be atomic */
   private final @NotNull Object sessionLock = new Object();
 
+  /** Scope's contexts */
+  private @NotNull Contexts contexts = new Contexts();
+
   /**
    * Scope's ctor
    *
@@ -125,7 +128,7 @@ List<String> getFingerprint() {
   }
 
   /**
-   * Sets the Scoope's fingerprint list
+   * Sets the Scope's fingerprint list
    *
    * @param fingerprint the fingerprint list
    */
@@ -278,6 +281,24 @@ public void removeExtra(@NotNull String key) {
     this.extra.remove(key);
   }
 
+  /**
+   * Returns the Scope's contexts
+   *
+   * @return the contexts
+   */
+  public @NotNull Contexts getContexts() {
+    return contexts;
+  }
+
+  /**
+   * Sets the Scope's contexts
+   *
+   * @param contexts the contexts
+   */
+  public void setContexts(final @NotNull Contexts contexts) {
+    this.contexts = contexts;
+  }
+
   /**
    * Creates a breadcrumb list with the max number of breadcrumbs
    *
@@ -332,7 +353,7 @@ public void removeExtra(@NotNull String key) {
 
     final Map<String, Object> extraRef = extra;
 
-    Map<String, Object> extraClone = new HashMap<>();
+    Map<String, Object> extraClone = new ConcurrentHashMap<>();
 
     for (Map.Entry<String, Object> item : extraRef.entrySet()) {
       if (item != null) {
@@ -342,6 +363,8 @@ public void removeExtra(@NotNull String key) {
 
     clone.extra = extraClone;
 
+    clone.contexts = contexts.clone();
+
     return clone;
   }
 

sentry-core/src/main/java/io/sentry/core/protocol/App.java

@@ -1,11 +1,14 @@
 package io.sentry.core.protocol;
 
 import io.sentry.core.IUnknownPropertiesConsumer;
+import io.sentry.core.util.CollectionUtils;
 import java.util.Date;
 import java.util.Map;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.TestOnly;
 
-public final class App implements IUnknownPropertiesConsumer {
+public final class App implements IUnknownPropertiesConsumer, Cloneable {
   public static final String TYPE = "app";
 
   private String appIdentifier;
@@ -77,9 +80,29 @@ public void setAppBuild(String appBuild) {
     this.appBuild = appBuild;
   }
 
+  @TestOnly
+  Map<String, Object> getUnknown() {
+    return unknown;
+  }
+
   @ApiStatus.Internal
   @Override
   public void acceptUnknownProperties(Map<String, Object> unknown) {
     this.unknown = unknown;
   }
+
+  /**
+   * Clones an App aka deep copy
+   *
+   * @return the cloned App
+   * @throws CloneNotSupportedException if object is not cloneable
+   */
+  @Override
+  public @NotNull App clone() throws CloneNotSupportedException {
+    final App clone = (App) super.clone();
+
+    clone.unknown = CollectionUtils.shallowCopy(unknown);
+
+    return clone;
+  }
 }

sentry-core/src/main/java/io/sentry/core/protocol/Browser.java

@@ -1,10 +1,13 @@
 package io.sentry.core.protocol;
 
 import io.sentry.core.IUnknownPropertiesConsumer;
+import io.sentry.core.util.CollectionUtils;
 import java.util.Map;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.TestOnly;
 
-public final class Browser implements IUnknownPropertiesConsumer {
+public final class Browser implements IUnknownPropertiesConsumer, Cloneable {
   public static final String TYPE = "browser";
   private String name;
   private String version;
@@ -28,9 +31,29 @@ public void setVersion(String version) {
     this.version = version;
   }
 
+  @TestOnly
+  Map<String, Object> getUnknown() {
+    return unknown;
+  }
+
   @ApiStatus.Internal
   @Override
   public void acceptUnknownProperties(Map<String, Object> unknown) {
     this.unknown = unknown;
   }
+
+  /**
+   * Clones a Browser aka deep copy
+   *
+   * @return the cloned Browser
+   * @throws CloneNotSupportedException if object is not cloneable
+   */
+  @Override
+  public @NotNull Browser clone() throws CloneNotSupportedException {
+    final Browser clone = (Browser) super.clone();
+
+    clone.unknown = CollectionUtils.shallowCopy(unknown);
+
+    return clone;
+  }
 }

sentry-core/src/main/java/io/sentry/core/protocol/Contexts.java

@@ -1,10 +1,21 @@
 package io.sentry.core.protocol;
 
+import java.util.Collections;
+import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
+import org.jetbrains.annotations.NotNull;
 
-public final class Contexts extends ConcurrentHashMap<String, Object> {
+public final class Contexts extends ConcurrentHashMap<String, Object> implements Cloneable {
   private static final long serialVersionUID = 252445813254943011L;
 
+  public Contexts(Map<String, Object> initialValues) {
+    super(initialValues);
+  }
+
+  public Contexts() {
+    this(Collections.emptyMap());
+  }
+
   private <T> T toContextType(String key, Class<T> clazz) {
     Object item = get(key);
     return clazz.isInstance(item) ? clazz.cast(item) : null;
@@ -57,4 +68,33 @@ public Gpu getGpu() {
   public void setGpu(Gpu gpu) {
     this.put(Gpu.TYPE, gpu);
   }
+
+  @Override
+  public @NotNull Contexts clone() throws CloneNotSupportedException {
+    final Contexts clone = new Contexts();
+
+    for (Map.Entry<String, Object> entry : entrySet()) {
+      if (entry != null) {
+        Object value = entry.getValue();
+        if (App.TYPE.equals(entry.getKey()) && value instanceof App) {
+          clone.setApp(((App) value).clone());
+        } else if (Browser.TYPE.equals(entry.getKey()) && value instanceof Browser) {
+          clone.setBrowser(((Browser) value).clone());
+        } else if (Device.TYPE.equals(entry.getKey()) && value instanceof Device) {
+          clone.setDevice(((Device) value).clone());
+        } else if (OperatingSystem.TYPE.equals(entry.getKey())
+            && value instanceof OperatingSystem) {
+          clone.setOperatingSystem(((OperatingSystem) value).clone());
+        } else if (SentryRuntime.TYPE.equals(entry.getKey()) && value instanceof SentryRuntime) {
+          clone.setRuntime(((SentryRuntime) value).clone());
+        } else if (Gpu.TYPE.equals(entry.getKey()) && value instanceof Gpu) {
+          clone.setGpu(((Gpu) value).clone());
+        } else {
+          clone.put(entry.getKey(), value);
+        }
+      }
+    }
+
+    return clone;
+  }
 }

sentry-core/src/main/java/io/sentry/core/protocol/Device.java

@@ -1,12 +1,15 @@
 package io.sentry.core.protocol;
 
 import io.sentry.core.IUnknownPropertiesConsumer;
+import io.sentry.core.util.CollectionUtils;
 import java.util.Date;
 import java.util.Map;
 import java.util.TimeZone;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.TestOnly;
 
-public final class Device implements IUnknownPropertiesConsumer {
+public final class Device implements IUnknownPropertiesConsumer, Cloneable {
   public static final String TYPE = "device";
 
   private String name;
@@ -343,9 +346,35 @@ public enum DeviceOrientation {
     LANDSCAPE
   }
 
+  @TestOnly
+  Map<String, Object> getUnknown() {
+    return unknown;
+  }
+
   @ApiStatus.Internal
   @Override
   public void acceptUnknownProperties(Map<String, Object> unknown) {
     this.unknown = unknown;
   }
+
+  /**
+   * Clones a Device aka deep copy
+   *
+   * @return the cloned Device
+   * @throws CloneNotSupportedException if object is not cloneable
+   */
+  @Override
+  public @NotNull Device clone() throws CloneNotSupportedException {
+    final Device clone = (Device) super.clone();
+
+    final String[] archsRef = this.archs;
+    clone.archs = archsRef != null ? this.archs.clone() : null;
+
+    final TimeZone timezoneRef = this.timezone;
+    clone.timezone = timezoneRef != null ? (TimeZone) this.timezone.clone() : null;
+
+    clone.unknown = CollectionUtils.shallowCopy(unknown);
+
+    return clone;
+  }
 }

sentry-core/src/main/java/io/sentry/core/protocol/Gpu.java

@@ -1,10 +1,13 @@
 package io.sentry.core.protocol;
 
 import io.sentry.core.IUnknownPropertiesConsumer;
+import io.sentry.core.util.CollectionUtils;
 import java.util.Map;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.TestOnly;
 
-public final class Gpu implements IUnknownPropertiesConsumer {
+public final class Gpu implements IUnknownPropertiesConsumer, Cloneable {
   public static final String TYPE = "gpu";
 
   private String name;
@@ -92,9 +95,29 @@ public void setNpotSupport(String npotSupport) {
     this.npotSupport = npotSupport;
   }
 
+  @TestOnly
+  Map<String, Object> getUnknown() {
+    return unknown;
+  }
+
   @ApiStatus.Internal
   @Override
   public void acceptUnknownProperties(Map<String, Object> unknown) {
     this.unknown = unknown;
   }
+
+  /**
+   * Clones a Gpu aka deep copy
+   *
+   * @return the cloned Gpu
+   * @throws CloneNotSupportedException if object is not cloneable
+   */
+  @Override
+  public @NotNull Gpu clone() throws CloneNotSupportedException {
+    final Gpu clone = (Gpu) super.clone();
+
+    clone.unknown = CollectionUtils.shallowCopy(unknown);
+
+    return clone;
+  }
 }

sentry-core/src/main/java/io/sentry/core/protocol/OperatingSystem.java

@@ -1,10 +1,13 @@
 package io.sentry.core.protocol;
 
 import io.sentry.core.IUnknownPropertiesConsumer;
+import io.sentry.core.util.CollectionUtils;
 import java.util.Map;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.NotNull;
+import org.jetbrains.annotations.TestOnly;
 
-public final class OperatingSystem implements IUnknownPropertiesConsumer {
+public final class OperatingSystem implements IUnknownPropertiesConsumer, Cloneable {
   public static final String TYPE = "os";
 
   private String name;
@@ -65,9 +68,29 @@ public void setRooted(Boolean rooted) {
     this.rooted = rooted;
   }
 
+  @TestOnly
+  Map<String, Object> getUnknown() {
+    return unknown;
+  }
+
   @ApiStatus.Internal
   @Override
   public void acceptUnknownProperties(Map<String, Object> unknown) {
     this.unknown = unknown;
   }
+
+  /**
+   * Clones an OperatingSystem aka deep copy
+   *
+   * @return the cloned OperatingSystem
+   * @throws CloneNotSupportedException if object is not cloneable
+   */
+  @Override
+  public @NotNull OperatingSystem clone() throws CloneNotSupportedException {
+    final OperatingSystem clone = (OperatingSystem) super.clone();
+
+    clone.unknown = CollectionUtils.shallowCopy(unknown);
+
+    return clone;
+  }
 }