feat(contexts): Add chipset to device context

Author: romtsn

sentry-android-core/src/main/java/io/sentry/android/core/DeviceInfoUtil.java

@@ -96,6 +96,7 @@ public static void resetInstance() {
 
   // we can get some inspiration here
   // https://github.com/flutter/plugins/blob/master/packages/device_info/android/src/main/java/io/flutter/plugins/deviceinfo/DeviceInfoPlugin.java
+  @SuppressLint("NewApi")
   @NotNull
   public Device collectDeviceInformation(
       final boolean collectDeviceIO, final boolean collectDynamicData) {
@@ -107,6 +108,9 @@ public Device collectDeviceInformation(
     device.setModel(Build.MODEL);
     device.setModelId(Build.ID);
     device.setArchs(ContextUtils.getArchitectures());
+    if (buildInfoProvider.getSdkInfoVersion() >= Build.VERSION_CODES.S) {
+      device.setChipset(Build.SOC_MANUFACTURER + " " + Build.SOC_MODEL);
+    }
 
     device.setOrientation(getOrientation());
     if (isEmulator != null) {

sentry/api/sentry.api

@@ -5155,6 +5155,7 @@ public final class io/sentry/protocol/Device : io/sentry/JsonSerializable, io/se
 	public fun getBatteryTemperature ()Ljava/lang/Float;
 	public fun getBootTime ()Ljava/util/Date;
 	public fun getBrand ()Ljava/lang/String;
+	public fun getChipset ()Ljava/lang/String;
 	public fun getConnectionType ()Ljava/lang/String;
 	public fun getCpuDescription ()Ljava/lang/String;
 	public fun getExternalFreeStorage ()Ljava/lang/Long;
@@ -5192,6 +5193,7 @@ public final class io/sentry/protocol/Device : io/sentry/JsonSerializable, io/se
 	public fun setBootTime (Ljava/util/Date;)V
 	public fun setBrand (Ljava/lang/String;)V
 	public fun setCharging (Ljava/lang/Boolean;)V
+	public fun setChipset (Ljava/lang/String;)V
 	public fun setConnectionType (Ljava/lang/String;)V
 	public fun setCpuDescription (Ljava/lang/String;)V
 	public fun setExternalFreeStorage (Ljava/lang/Long;)V
@@ -5249,6 +5251,7 @@ public final class io/sentry/protocol/Device$JsonKeys {
 	public static final field BOOT_TIME Ljava/lang/String;
 	public static final field BRAND Ljava/lang/String;
 	public static final field CHARGING Ljava/lang/String;
+	public static final field CHIPSET Ljava/lang/String;
 	public static final field CONNECTION_TYPE Ljava/lang/String;
 	public static final field CPU_DESCRIPTION Ljava/lang/String;
 	public static final field EXTERNAL_FREE_STORAGE Ljava/lang/String;

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

@@ -124,6 +124,8 @@ public final class Device implements JsonUnknown, JsonSerializable {
   /** Optional. CPU description. For example, Intel(R) Core(TM)2 Quad CPU Q6600 @ 2.40GHz. */
   private @Nullable String cpuDescription;
 
+  private @Nullable String chipset;
+
   @SuppressWarnings("unused")
   private @Nullable Map<String, @NotNull Object> unknown;
 
@@ -167,6 +169,7 @@ public Device() {}
     this.processorCount = device.processorCount;
     this.processorFrequency = device.processorFrequency;
     this.cpuDescription = device.cpuDescription;
+    this.chipset = device.chipset;
 
     this.unknown = CollectionUtils.newConcurrentHashMap(device.unknown);
   }
@@ -429,6 +432,14 @@ public void setCpuDescription(@Nullable final String cpuDescription) {
     this.cpuDescription = cpuDescription;
   }
 
+  public @Nullable String getChipset() {
+    return chipset;
+  }
+
+  public void setChipset(final @Nullable String chipset) {
+    this.chipset = chipset;
+  }
+
   @Override
   public boolean equals(Object o) {
     if (this == o) return true;
@@ -465,7 +476,8 @@ public boolean equals(Object o) {
         && Objects.equals(batteryTemperature, device.batteryTemperature)
         && Objects.equals(processorCount, device.processorCount)
         && Objects.equals(processorFrequency, device.processorFrequency)
-        && Objects.equals(cpuDescription, device.cpuDescription);
+        && Objects.equals(cpuDescription, device.cpuDescription)
+        && Objects.equals(chipset, device.chipset);
   }
 
   @Override
@@ -503,7 +515,8 @@ public int hashCode() {
             batteryTemperature,
             processorCount,
             processorFrequency,
-            cpuDescription);
+            cpuDescription,
+            chipset);
     result = 31 * result + Arrays.hashCode(archs);
     return result;
   }
@@ -567,6 +580,7 @@ public static final class JsonKeys {
     public static final String PROCESSOR_COUNT = "processor_count";
     public static final String CPU_DESCRIPTION = "cpu_description";
     public static final String PROCESSOR_FREQUENCY = "processor_frequency";
+    public static final String CHIPSET = "chipset";
   }
 
   @Override
@@ -672,6 +686,9 @@ public void serialize(final @NotNull ObjectWriter writer, final @NotNull ILogger
     if (cpuDescription != null) {
       writer.name(JsonKeys.CPU_DESCRIPTION).value(cpuDescription);
     }
+    if (chipset != null) {
+      writer.name(JsonKeys.CHIPSET).value(chipset);
+    }
     if (unknown != null) {
       for (String key : unknown.keySet()) {
         Object value = unknown.get(key);
@@ -817,6 +834,9 @@ public static final class Deserializer implements JsonDeserializer<Device> {
           case JsonKeys.CPU_DESCRIPTION:
             device.cpuDescription = reader.nextStringOrNull();
             break;
+          case JsonKeys.CHIPSET:
+            device.chipset = reader.nextStringOrNull();
+            break;
           default:
             if (unknown == null) {
               unknown = new ConcurrentHashMap<>();

sentry/src/test/java/io/sentry/protocol/DeviceSerializationTest.kt

@@ -56,6 +56,7 @@ class DeviceSerializationTest {
             connectionType = "9ceb3a6c-5292-4ed9-8665-5732495e8ed4"
             batteryTemperature = 0.14775127f
             cpuDescription = "cpu0"
+            chipset = "unisoc t606"
             processorCount = 4
             processorFrequency = 800.0
         }

sentry/src/test/java/io/sentry/protocol/DeviceTest.kt

@@ -62,6 +62,7 @@ class DeviceTest {
         device.batteryTemperature = 30f
         device.locale = "en-US"
         device.cpuDescription = "cpu0"
+        device.chipset = "unisoc t606"
         device.processorCount = 4
         device.processorFrequency = 800.0
         val unknown = mapOf(Pair("unknown", "unknown"))
@@ -101,6 +102,7 @@ class DeviceTest {
         assertEquals("connection type", clone.connectionType)
         assertEquals(30f, clone.batteryTemperature)
         assertEquals("cpu0", clone.cpuDescription)
+        assertEquals("unisoc t606", clone.chipset)
         assertEquals(4, clone.processorCount)
         assertEquals(800.0, clone.processorFrequency)
         device.processorFrequency = 800.0

sentry/src/test/resources/json/device.json

@@ -38,5 +38,6 @@
     "battery_temperature": 0.14775127,
     "processor_count": 4,
     "processor_frequency": 800.0,
-    "cpu_description": "cpu0"
+    "cpu_description": "cpu0",
+    "chipset": "unisoc t606"
 }