blacklist some devices with buggy MediaCodec support

media/base/android/media_codec_util.cc

Index: media/base/android/media_codec_util.cc
diff --git a/media/base/android/media_codec_util.cc b/media/base/android/media_codec_util.cc
index c01aabe826be829edcca7d7a4cc1461b2bdccdc9..a776dd139804f3edb6f470856d4a38ec5a04048a 100644
--- a/media/base/android/media_codec_util.cc
+++ b/media/base/android/media_codec_util.cc
@@ -89,11 +89,27 @@ bool MediaCodecUtil::IsMediaCodecAvailable() {
   // MediaCodec is only available on JB and greater.
   if (base::android::BuildInfo::GetInstance()->sdk_int() < 16)
     return false;
+
+  // Blacklist some devices on Jellybean-MR2 as for MediaCodec support is buggy.
+  // http://crbug.com/615872
+  if (base::android::BuildInfo::GetInstance()->sdk_int() == 18) {

[liberato (no reviews please), 2016/05/31 15:37:08]

i'd be surprised if api level 17 (JB MR1) weren't also broken.

please merge all three conditionals into one check for api <= 19.  it's probably
also safe to include the N7000 in there, unless you know that it works on > 16. 
that might be the most recent official update anyway.

+    std::string model(base::android::BuildInfo::GetInstance()->model());
+    return model != "GT-I9100" && model != "GT-I9300" && model != "GT-N7100";
+  }
+
+  // Blacklist some devices on KitKat as for MediaCodec support is buggy.
+  // http://crbug.com/615872
+  if (base::android::BuildInfo::GetInstance()->sdk_int() == 19) {
+    std::string model(base::android::BuildInfo::GetInstance()->model());
+    return model != "GT-I9100" && model != "GT-I9300" && model != "GT-N7100";
+  }
+
   // Blacklist some devices on Jellybean as for MediaCodec support is buggy.
   // http://crbug.com/365494.
   if (base::android::BuildInfo::GetInstance()->sdk_int() == 16) {
     std::string model(base::android::BuildInfo::GetInstance()->model());
-    return model != "GT-I9100" && model != "GT-I9300" && model != "GT-N7000";
+    return model != "GT-I9100" && model != "GT-I9300" && model != "GT-N7000"
+        && model != "GT-N7100";
   }
   return true;
 }