Merge pull request #5891 from element-hq/feature/bma/qrCodeScannerCleanup

Qr code scanner cleanup

features/login/impl/src/main/kotlin/io/element/android/features/login/impl/screens/qrcode/scan/QrCodeScanView.kt

@@ -106,7 +106,7 @@ private fun Content(
             QrCodeCameraView(
                 modifier = Modifier.fillMaxSize(),
                 onScanQrCode = { state.eventSink.invoke(QrCodeScanEvents.QrCodeScanned(it)) },
-                renderPreview = state.isScanning,
+                isScanning = state.isScanning,
             )
         }
     }

libraries/qrcode/src/main/kotlin/io/element/android/libraries/qrcode/QRCodeAnalyzer.kt

@@ -15,19 +15,24 @@ import timber.log.Timber
 import zxingcpp.BarcodeReader
 
 internal class QRCodeAnalyzer(
-    private val onScanQrCode: (result: ByteArray?) -> Unit
+    private val onScanQrCode: (data: ByteArray) -> Unit
 ) : ImageAnalysis.Analyzer {
     private val reader by lazy { BarcodeReader() }
 
     override fun analyze(image: ImageProxy) {
+        image.use {
             if (image.format in SUPPORTED_IMAGE_FORMATS) {
                 try {
                     val bytes = reader.read(image).firstNotNullOfOrNull { it.bytes }
-                bytes?.let { onScanQrCode(it) }
+                    if (bytes != null) {
+                        Timber.d("QR code scanned!")
+                        onScanQrCode(bytes)
+                    }
                 } catch (e: Exception) {
                     Timber.w(e, "Error decoding QR code")
-            } finally {
+                }
-                image.close()
+            } else {
+                Timber.w("Unsupported image format: ${image.format}")
             }
         }
     }

libraries/qrcode/src/main/kotlin/io/element/android/libraries/qrcode/QrCodeCameraView.kt

@@ -45,18 +45,9 @@ import kotlin.coroutines.suspendCoroutine
 @Composable
 fun QrCodeCameraView(
     onScanQrCode: (ByteArray) -> Unit,
-    renderPreview: Boolean,
+    isScanning: Boolean,
     modifier: Modifier = Modifier,
 ) {
-    if (LocalInspectionMode.current) {
-        Box(
-            modifier = modifier
-                .background(color = ElementTheme.colors.bgSubtlePrimary),
-            contentAlignment = Alignment.Center,
-        ) {
-            Text("CameraView")
-        }
-    } else {
     val coroutineScope = rememberCoroutineScope()
     val localContext = LocalContext.current
     val lifecycleOwner = LocalLifecycleOwner.current
@@ -73,19 +64,14 @@ fun QrCodeCameraView(
         cameraProvider = localContext.getCameraProvider()
     }
 
-        suspend fun startQRCodeAnalysis(cameraProvider: ProcessCameraProvider, previewView: PreviewView, attempt: Int = 1) {
+    suspend fun startQRCodeAnalysis(cameraProvider: ProcessCameraProvider, attempt: Int = 1) {
         lastFrame = null
         val cameraSelector = CameraSelector.Builder()
             .requireLensFacing(CameraSelector.LENS_FACING_BACK)
             .build()
         imageAnalysis.setAnalyzer(
-                ContextCompat.getMainExecutor(previewView.context),
+            ContextCompat.getMainExecutor(localContext),
-                QRCodeAnalyzer { result ->
+            QRCodeAnalyzer(onScanQrCode)
-                    result?.let {
-                        Timber.d("QR code scanned!")
-                        onScanQrCode(it)
-                    }
-                }
         )
         try {
             // Make sure we unbind all use cases before binding them again
@@ -95,7 +81,7 @@ fun QrCodeCameraView(
                 lifecycleOwner,
                 cameraSelector,
                 previewUseCase,
-                    imageAnalysis
+                imageAnalysis,
             )
             lastFrame = null
         } catch (e: Exception) {
@@ -105,7 +91,7 @@ fun QrCodeCameraView(
             } else {
                 Timber.e(e, "Use case binding failed (attempt #$attempt). Retrying after a delay...")
                 delay(100)
-                    startQRCodeAnalysis(cameraProvider, previewView, attempt + 1)
+                startQRCodeAnalysis(cameraProvider, attempt + 1)
             }
         }
     }
@@ -125,19 +111,30 @@ fun QrCodeCameraView(
     }
 
     Box(modifier.clipToBounds()) {
+        if (LocalInspectionMode.current) {
+            Box(
+                modifier = modifier
+                    .background(color = ElementTheme.colors.bgSubtlePrimary),
+                contentAlignment = Alignment.Center,
+            ) {
+                Text("CameraView")
+            }
+        } else {
             AndroidView(
                 factory = { context ->
                     val previewView = PreviewView(context)
-                    previewUseCase.setSurfaceProvider(previewView.surfaceProvider)
+                    previewUseCase.surfaceProvider = previewView.surfaceProvider
                     previewView.previewStreamState.observe(lifecycleOwner) { state ->
                         previewView.alpha = if (state == PreviewView.StreamState.STREAMING) 1f else 0f
                     }
                     previewView
                 },
                 update = { previewView ->
-                    if (renderPreview) {
+                    if (isScanning) {
                         cameraProvider?.let { provider ->
-                            coroutineScope.launch { startQRCodeAnalysis(provider, previewView) }
+                            coroutineScope.launch {
+                                startQRCodeAnalysis(provider)
+                            }
                         }
                     } else {
                         stopQRCodeAnalysis(previewView)
@@ -148,19 +145,21 @@ fun QrCodeCameraView(
                     cameraProvider = null
                 },
             )
+        }
         lastFrame?.let {
             Image(bitmap = it.asImageBitmap(), contentDescription = null)
         }
     }
 }
-}
 
-@Suppress("BlockingMethodInNonBlockingContext")
 private suspend fun Context.getCameraProvider(): ProcessCameraProvider =
     suspendCoroutine { continuation ->
         ProcessCameraProvider.getInstance(this).also { cameraProvider ->
-            cameraProvider.addListener({
+            cameraProvider.addListener(
+                {
                     continuation.resume(cameraProvider.get())
-            }, ContextCompat.getMainExecutor(this))
+                },
+                ContextCompat.getMainExecutor(this),
+            )
         }
     }