From 52337a094758a62085a977407bcde4ddcdaa01b6 Mon Sep 17 00:00:00 2001
From: Crepieux Pierre <pierre.crepieux@orange.com>
Date: Thu, 27 May 2021 17:07:17 +0200
Subject: [PATCH 1/2] Fix image size of kotlin minicap

The image reader was inadvertently initialized with the screen size
rather than the targeted size

Signed-off-by: Crepieux Pierre <pierre.crepieux@orange.com>
---
 .../src/main/java/io/devicefarmer/minicap/Main.kt | 15 ++++++++++++---
 .../devicefarmer/minicap/provider/BaseProvider.kt |  8 ++++++--
 2 files changed, 18 insertions(+), 5 deletions(-)

diff --git a/experimental/app/src/main/java/io/devicefarmer/minicap/Main.kt b/experimental/app/src/main/java/io/devicefarmer/minicap/Main.kt
index c8b8a6d..9293702 100644
--- a/experimental/app/src/main/java/io/devicefarmer/minicap/Main.kt
+++ b/experimental/app/src/main/java/io/devicefarmer/minicap/Main.kt
@@ -18,8 +18,7 @@ package io.devicefarmer.minicap
 import android.os.Looper
 import android.util.Size
 import io.devicefarmer.minicap.provider.SurfaceProvider
-import org.slf4j.LoggerFactory
-import java.io.PrintStream
+import kotlin.math.roundToInt
 import kotlin.system.exitProcess
 
 
@@ -58,6 +57,7 @@ class Main {
             provider = if (params.projection == null) {
                 SurfaceProvider()
             } else {
+                params.projection.forceAspectRatio()
                 SurfaceProvider(
                     Size(
                         params.projection.targetSize.width,
@@ -106,9 +106,18 @@ class Main {
 }
 
 data class Projection(
-    val realSize: Size, val targetSize: Size,
+    val realSize: Size, var targetSize: Size,
     val orientation: Int
 ) {
+    fun forceAspectRatio() {
+        val aspect = realSize.width.toFloat() / realSize.height.toFloat()
+        targetSize = if (targetSize.height > targetSize.width / aspect) {
+            Size(targetSize.width, ((targetSize.width / aspect)).roundToInt())
+        } else {
+            Size((targetSize.height * aspect).roundToInt(), targetSize.height)
+        }
+    }
+
     override fun toString(): String =
         "${realSize.width}x${realSize.height}@${targetSize.width}x${targetSize.height}/${orientation}"
 }
diff --git a/experimental/app/src/main/java/io/devicefarmer/minicap/provider/BaseProvider.kt b/experimental/app/src/main/java/io/devicefarmer/minicap/provider/BaseProvider.kt
index a90a865..617b56d 100644
--- a/experimental/app/src/main/java/io/devicefarmer/minicap/provider/BaseProvider.kt
+++ b/experimental/app/src/main/java/io/devicefarmer/minicap/provider/BaseProvider.kt
@@ -65,8 +65,8 @@ abstract class BaseProvider(private val targetSize: Size) : SimpleServer.Listene
 
     fun init(out: DisplayOutput) {
         imageReader = ImageReader.newInstance(
-            getScreenSize().width,
-            getScreenSize().height,
+            getTargetSize().width,
+            getTargetSize().height,
             PixelFormat.RGBA_8888,
             2
         )
@@ -111,6 +111,10 @@ abstract class BaseProvider(private val targetSize: Size) : SimpleServer.Listene
                 Bitmap.Config.ARGB_8888
             ).apply {
                 copyPixelsFromBuffer(buffer)
+            }.run {
+                //the image need to be cropped
+                Bitmap.createBitmap(this, 0, 0, getTargetSize().width, getTargetSize().height)
+            }.apply {
                 compress(Bitmap.CompressFormat.JPEG, q, out)
             }
         }

From 16243890b78fb4bd33c822ab0e71f6162bb6cd71 Mon Sep 17 00:00:00 2001
From: Crepieux Pierre <pierre.crepieux@orange.com>
Date: Thu, 27 May 2021 17:16:43 +0200
Subject: [PATCH 2/2] update apk version code / version name and npm package

Signed-off-by: Crepieux Pierre <pierre.crepieux@orange.com>
---
 experimental/app/build.gradle | 4 ++--
 package.json                  | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/experimental/app/build.gradle b/experimental/app/build.gradle
index 926f8d4..efff823 100644
--- a/experimental/app/build.gradle
+++ b/experimental/app/build.gradle
@@ -27,8 +27,8 @@ android {
         applicationId "io.devicefarmer.minicap"
         minSdkVersion 21
         targetSdkVersion 30
-        versionCode 1
-        versionName "0.3"
+        versionCode 2
+        versionName "0.7"
         archivesBaseName = "minicap"
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
     }
diff --git a/package.json b/package.json
index 995870d..6eb0b50 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@devicefarmer/minicap-prebuilt",
-  "version": "2.6.1",
+  "version": "2.7.0",
   "description": "Prebuilt binaries of minicap.",
   "keywords": [
     "minicap"