Bug 1362918 - Extend the blacklist to cover certain Galaxy S4 models. r?jolin draft
authorJames Cheng <jacheng@mozilla.com>
Mon, 08 May 2017 11:56:56 +0800
changeset 575191 7762421475840af51cd1713a31c4c0f974789666
parent 573891 c3e5497cff1c995821b1c9320fa71f1ef9a8c30e
child 627854 0fa7d8ca0666516102f31fb439e9d9c3c58413bf
push id57989
push userbmo:jacheng@mozilla.com
push dateWed, 10 May 2017 03:24:39 +0000
reviewersjolin
bugs1362918
milestone55.0a1
Bug 1362918 - Extend the blacklist to cover certain Galaxy S4 models. r?jolin MozReview-Commit-ID: WZFON3SBvU
mobile/android/geckoview/src/main/java/org/mozilla/gecko/util/HardwareCodecCapabilityUtils.java
--- a/mobile/android/geckoview/src/main/java/org/mozilla/gecko/util/HardwareCodecCapabilityUtils.java
+++ b/mobile/android/geckoview/src/main/java/org/mozilla/gecko/util/HardwareCodecCapabilityUtils.java
@@ -36,18 +36,22 @@ public final class HardwareCodecCapabili
   private static final int[] supportedColorList = {
     CodecCapabilities.COLOR_FormatYUV420Planar,
     CodecCapabilities.COLOR_FormatYUV420SemiPlanar,
     CodecCapabilities.COLOR_QCOM_FormatYUV420SemiPlanar,
     COLOR_QCOM_FORMATYUV420PackedSemiPlanar32m
   };
   private static final String[] adaptivePlaybackBlacklist =
   {
-    "GT-I9300", // S3 (I9300 / I9300I)
-    "SCH-I535"  // S3
+    "GT-I9300",         // S3 (I9300 / I9300I)
+    "SCH-I535",         // S3
+    "SGH-M919",         // S4
+    "GT-I9505",         // S4
+    "SGH-I337",         // S4
+    "SAMSUNG-SGH-I337"  // S4
   };
 
   @WrapForJNI
   public static boolean findDecoderCodecInfoForMimeType(String aMimeType) {
     int numCodecs = 0;
     try {
       numCodecs = MediaCodecList.getCodecCount();
     } catch (final RuntimeException e) {
@@ -87,24 +91,21 @@ public final class HardwareCodecCapabili
       Log.e(LOGTAG, "Retrieve codec information failed", e);
     }
     return false;
   }
 
   // See Bug1360626 and
   // https://codereview.chromium.org/1869103002 for details.
   private static boolean isAdaptivePlaybackBlacklisted(String aMimeType) {
+    Log.d(LOGTAG, "The device ModelID is " + Build.MODEL);
     if (!aMimeType.equals("video/avc") && !aMimeType.equals("video/avc1")) {
       return false;
     }
 
-    if (!Build.VERSION.RELEASE.equals("4.4.2")) {
-      return false;
-    }
-
     if (!Build.MANUFACTURER.toLowerCase(Locale.getDefault()).equals("samsung")) {
       return false;
     }
 
     for (String model : adaptivePlaybackBlacklist) {
       if (Build.MODEL.startsWith(model)) {
         return true;
       }