Increase frame capture quality, apply scaling after DPI normalization (#55731)

Summary:
Pull Request resolved: #55731

Updates Android screenshot capture for performance frame timings to improve image quality:

- Fix scale factor calculation by first normalizing output size based on display density.
- Increase (in most cases) output image size and encoding quality.

**Impact**

- Output images are now net larger (depending on DPI), and have significantly more clarity.
- Each captured frame now uses **~9x more space** (vs a worst case device scaling starting point).

NOTE: These changes come with a memory tradeoff, and we may explore multi-frame sampling and a bounded buffer in future. Not a major bottleneck, even with this size increase (the bottleneck remains in expanding JS Sampling Profiler events).

Changelog: [Internal]

Reviewed By: rubennorte

Differential Revision: D94256691

fbshipit-source-id: 91f3610f5a5ac10a9806c07c21c38b30599b93de

Author: huntie

packages/react-native/ReactAndroid/src/main/java/com/facebook/react/devsupport/inspector/FrameTimingsObserver.kt

@@ -124,14 +124,11 @@ internal class FrameTimingsObserver(
 
     // Reuse bitmap if dimensions haven't changed
     val bitmap =
-        bitmapBuffer?.let {
-          if (it.width == width && it.height == height) {
-            it
-          } else {
-            it.recycle()
-            null
-          }
-        } ?: Bitmap.createBitmap(width, height, Bitmap.Config.ARGB_8888).also { bitmapBuffer = it }
+        bitmapBuffer?.takeIf { it.width == width && it.height == height }
+            ?: run {
+              bitmapBuffer?.recycle()
+              Bitmap.createBitmap(width, height, Bitmap.Config.ARGB_8888).also { bitmapBuffer = it }
+            }
 
     PixelCopy.request(
         window,
@@ -141,19 +138,19 @@ internal class FrameTimingsObserver(
             CoroutineScope(Dispatchers.Default).launch {
               var scaledBitmap: Bitmap? = null
               try {
-                val scaleFactor = 0.25f
-                val scaledWidth = (width * scaleFactor).toInt()
-                val scaledHeight = (height * scaleFactor).toInt()
+                val density = window.context.resources.displayMetrics.density
+                val scaledWidth = (width / density * SCREENSHOT_SCALE_FACTOR).toInt()
+                val scaledHeight = (height / density * SCREENSHOT_SCALE_FACTOR).toInt()
                 scaledBitmap = Bitmap.createScaledBitmap(bitmap, scaledWidth, scaledHeight, true)
 
                 val compressFormat =
                     if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.R)
                         Bitmap.CompressFormat.WEBP_LOSSY
-                    else Bitmap.CompressFormat.WEBP
+                    else Bitmap.CompressFormat.JPEG
 
                 val base64 =
                     ByteArrayOutputStream().use { outputStream ->
-                      scaledBitmap.compress(compressFormat, 0, outputStream)
+                      scaledBitmap.compress(compressFormat, SCREENSHOT_QUALITY, outputStream)
                       Base64.encodeToString(outputStream.toByteArray(), Base64.NO_WRAP)
                     }
 
@@ -171,4 +168,9 @@ internal class FrameTimingsObserver(
         handler,
     )
   }
+
+  companion object {
+    private const val SCREENSHOT_SCALE_FACTOR = 0.75f
+    private const val SCREENSHOT_QUALITY = 80
+  }
 }