[go: nahoru, domu]

Merge "Exclude FHD and UHD video quality for Samsung J4." into androidx-main
diff --git a/camera/camera-video/src/main/java/androidx/camera/video/VideoCapabilities.java b/camera/camera-video/src/main/java/androidx/camera/video/VideoCapabilities.java
index ecc33f6..e39a21c 100644
--- a/camera/camera-video/src/main/java/androidx/camera/video/VideoCapabilities.java
+++ b/camera/camera-video/src/main/java/androidx/camera/video/VideoCapabilities.java
@@ -30,6 +30,7 @@
 import androidx.camera.core.impl.CameraInfoInternal;
 import androidx.camera.core.impl.utils.CompareSizesByArea;
 import androidx.camera.video.internal.compat.quirk.DeviceQuirks;
+import androidx.camera.video.internal.compat.quirk.ExcludeStretchedVideoQualityQuirk;
 import androidx.camera.video.internal.compat.quirk.VideoEncoderCrashQuirk;
 import androidx.camera.video.internal.compat.quirk.VideoQualityNotSupportQuirk;
 import androidx.camera.video.internal.compat.quirk.VideoQualityQuirk;
@@ -202,7 +203,8 @@
 
     private boolean isDeviceValidQuality(@NonNull Quality quality) {
         List<Class<? extends VideoQualityQuirk>> quirkList = Arrays.asList(
-                VideoQualityNotSupportQuirk.class, VideoEncoderCrashQuirk.class);
+                ExcludeStretchedVideoQualityQuirk.class, VideoQualityNotSupportQuirk.class,
+                VideoEncoderCrashQuirk.class);
 
         for (Class<? extends VideoQualityQuirk> quirkClass : quirkList) {
             VideoQualityQuirk quirk = DeviceQuirks.get(quirkClass);
diff --git a/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/DeviceQuirksLoader.java b/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/DeviceQuirksLoader.java
index d5e9895..12d4dec 100644
--- a/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/DeviceQuirksLoader.java
+++ b/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/DeviceQuirksLoader.java
@@ -62,6 +62,9 @@
         if (VideoEncoderCrashQuirk.load()) {
             quirks.add(new VideoEncoderCrashQuirk());
         }
+        if (ExcludeStretchedVideoQualityQuirk.load()) {
+            quirks.add(new ExcludeStretchedVideoQualityQuirk());
+        }
 
         return quirks;
     }
diff --git a/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/ExcludeStretchedVideoQualityQuirk.java b/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/ExcludeStretchedVideoQualityQuirk.java
new file mode 100644
index 0000000..7128625
--- /dev/null
+++ b/camera/camera-video/src/main/java/androidx/camera/video/internal/compat/quirk/ExcludeStretchedVideoQualityQuirk.java
@@ -0,0 +1,49 @@
+/*
+ * Copyright 2022 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package androidx.camera.video.internal.compat.quirk;
+
+import android.os.Build;
+
+import androidx.annotation.NonNull;
+import androidx.annotation.RequiresApi;
+import androidx.camera.video.Quality;
+
+/**
+ * Bug Id: 202792648
+ * Description: The captured video is stretched while selecting the quality is greater or
+ * equality to FHD resolution.
+ * Device(s): Samsung J4 (sm-j400g)
+ */
+@RequiresApi(21) // TODO(b/200306659): Remove and replace with annotation on package-info.java
+public class ExcludeStretchedVideoQualityQuirk implements VideoQualityQuirk {
+    static boolean load() {
+        return isSamsungJ4();
+    }
+
+    private static boolean isSamsungJ4() {
+        return "Samsung".equalsIgnoreCase(Build.BRAND) && "SM-J400G".equalsIgnoreCase(Build.MODEL);
+    }
+
+    /** Checks if the given Quality type is a problematic quality. */
+    @Override
+    public boolean isProblematicVideoQuality(@NonNull Quality quality) {
+        if (isSamsungJ4()) {
+            return quality == Quality.FHD || quality == Quality.UHD;
+        }
+        return false;
+    }
+}