match unit test

Author: abelonogov-ld

sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/ReplayOptions.kt

@@ -23,7 +23,7 @@ data class ReplayOptions(
     sealed class CompressionMethod {
         data object ScreenImage : CompressionMethod()
         data class OverlayTiles(
-            val layers: Int = 10,
+            val layers: Int = 15,
             val backtracking: Boolean = true,
         ) : CompressionMethod()
     }

sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/capture/ExportDiffManager.kt

@@ -46,39 +46,35 @@ class ExportDiffManager(
                     removes = currentImages.subList(lastKeyNodeIdx + 1, currentImages.size).toMutableList()
                     currentImages.subList(lastKeyNodeIdx + 1, currentImages.size).clear()
 
-                    // Keep only signatures that still point into the retained prefix.
                     val filtered = currentImagesIndex.filterValues { value -> value <= lastKeyNodeIdx }
                     currentImagesIndex.clear()
                     currentImagesIndex.putAll(filtered)
                 } else {
-                    for ((tileIdx, tile) in tiledFrame.tiles.withIndex()) {
-                        val tileSignature = signature.tileSignatures.getOrNull(tileIdx)
+                    for (tile in tiledFrame.tiles) {
                         val addImage =
-                            tile.bitmap.asExportedImage(format = format, rect = tile.rect, tileSignature = tileSignature)
+                            tile.bitmap.asExportedImage(format = format, rect = tile.rect, imageSignature = signature)
                                 ?: return null
                         adds.add(addImage)
-                        if (tileSignature != null) {
-                            currentImages.add(
-                                ExportFrame.RemoveImage(
-                                    keyFrameId = keyFrameId,
-                                    tileSignature = tileSignature,
-                                )
+                        currentImages.add(
+                            ExportFrame.RemoveImage(
+                                keyFrameId = keyFrameId,
+                                imageSignature = signature,
                             )
-                        }
+                        )
                     }
                     currentImagesIndex[signature] = currentImages.size - 1
                 }
             } else {
-                for ((tileIdx, tile) in tiledFrame.tiles.withIndex()) {
-                    val tileSignature = signature?.tileSignatures?.getOrNull(tileIdx)
-                    val addImage = tile.bitmap.asExportedImage(format = format, rect = tile.rect, tileSignature = tileSignature)
+                for (tile in tiledFrame.tiles) {
+                    val imageSignature = tiledFrame.imageSignature
+                    val addImage = tile.bitmap.asExportedImage(format = format, rect = tile.rect, imageSignature = imageSignature)
                         ?: return null
                     adds.add(addImage)
-                    if (tileSignature != null) {
+                    if (imageSignature != null) {
                         currentImages.add(
                             ExportFrame.RemoveImage(
                                 keyFrameId = keyFrameId,
-                                tileSignature = tileSignature,
+                                imageSignature = imageSignature,
                             )
                         )
                     }
@@ -114,7 +110,7 @@ class ExportDiffManager(
 private fun Bitmap.asExportedImage(
     format: ExportFrame.ExportFormat,
     rect: IntRect,
-    tileSignature: TileSignature?,
+    imageSignature: ImageSignature?,
 ): ExportFrame.AddImage? {
     val outputStream = ByteArrayOutputStream()
     return try {
@@ -138,7 +134,7 @@ private fun Bitmap.asExportedImage(
         ExportFrame.AddImage(
             imageBase64 = compressedImage,
             rect = rect,
-            tileSignature = tileSignature,
+            imageSignature = imageSignature,
         )
     } finally {
         outputStream.close()

sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/capture/ExportFrame.kt

@@ -28,7 +28,7 @@ data class ExportFrame(
             AddImage(
                 imageBase64 = imageBase64,
                 rect = IntRect(left = 0, top = 0, width = origWidth, height = origHeight),
-                tileSignature = null
+                imageSignature = null
             )
         ),
         removeImages = null,
@@ -44,13 +44,13 @@ data class ExportFrame(
 
     data class RemoveImage(
         val keyFrameId: Int,
-        val tileSignature: TileSignature,
+        val imageSignature: ImageSignature,
     )
 
     data class AddImage(
         val imageBase64: String,
         val rect: IntRect,
-        val tileSignature: TileSignature?,
+        val imageSignature: ImageSignature?,
     )
 
     sealed class ExportFormat {

sdk/@launchdarkly/observability-android/lib/src/main/kotlin/com/launchdarkly/observability/replay/exporter/RRWebEventGenerator.kt

@@ -15,7 +15,7 @@ import com.launchdarkly.observability.replay.RRWebCustomDataTag
 import com.launchdarkly.observability.replay.RRWebIncrementalSource
 import com.launchdarkly.observability.replay.RRWebMouseInteraction
 import com.launchdarkly.observability.replay.capture.ExportFrame
-import com.launchdarkly.observability.replay.capture.TileSignature
+import com.launchdarkly.observability.replay.capture.ImageSignature
 import kotlinx.serialization.json.JsonPrimitive
 import kotlinx.serialization.json.buildJsonArray
 import kotlinx.serialization.json.buildJsonObject
@@ -57,7 +57,7 @@ class RRWebEventGenerator(
     private var imageNodeId: Int? = null
     private var bodyNodeId: Int? = null
     private var knownKeyFrameId: Int? = null
-    private val nodeIds = mutableMapOf<TileSignature, Int>()
+    private val nodeIds = mutableMapOf<ImageSignature, Int>()
 
     data class State(
         val lastSid: Int = 0,
@@ -66,7 +66,7 @@ class RRWebEventGenerator(
         val imageNodeId: Int? = null,
         val bodyNodeId: Int? = null,
         val knownKeyFrameId: Int? = null,
-        val nodeIds: Map<TileSignature, Int> = emptyMap(),
+        val nodeIds: Map<ImageSignature, Int> = emptyMap(),
     )
 
     private fun nextSid(): Int {
@@ -88,7 +88,7 @@ class RRWebEventGenerator(
 
     private fun tileNode(exportFrame: ExportFrame, image: ExportFrame.AddImage): Pair<EventNode, Int> {
         val tileCanvasId = nextNodeId()
-        image.tileSignature?.let { nodeIds[it] = tileCanvasId }
+        image.imageSignature?.let { nodeIds[it] = tileCanvasId }
         val dataUrl = "data:${imageMimeType(exportFrame.format)};base64,${image.imageBase64}"
         val node = EventNode(
             id = tileCanvasId,
@@ -110,7 +110,7 @@ class RRWebEventGenerator(
 
         var totalCanvasSize = 0
         val removes = exportFrame.removeImages?.mapNotNull { removal ->
-            nodeIds[removal.tileSignature]?.let { nodeId ->
+            nodeIds[removal.imageSignature]?.let { nodeId ->
                 Removal(parentId = bodyId, id = nodeId)
             }
         } ?: emptyList()