Blacklist some samsung devices from MSE support

media/base/android/media_codec_bridge.cc

Index: media/base/android/media_codec_bridge.cc
diff --git a/media/base/android/media_codec_bridge.cc b/media/base/android/media_codec_bridge.cc
index 7ece5d558e6fa03937e5efc3b6fb1a1d5cbcccfb..7b8b7f49d1063c0a111f3728fbab132f973f2757 100644
--- a/media/base/android/media_codec_bridge.cc
+++ b/media/base/android/media_codec_bridge.cc
@@ -104,7 +104,14 @@ ToJavaIntArray(JNIEnv* env, scoped_ptr<jint[]> native_array, int size) {
 // static
 bool MediaCodecBridge::IsAvailable() {
   // MediaCodec is only available on JB and greater.
-  return base::android::BuildInfo::GetInstance()->sdk_int() >= 16;
+  if (base::android::BuildInfo::GetInstance()->sdk_int() < 16)
+    return false;
+  if (base::android::BuildInfo::GetInstance()->sdk_int() == 16) {
+    std::string model(base::android::BuildInfo::GetInstance()->model());
+    return model.compare("GT-I9100") && model.compare("GT-I9300") &&
+        model.compare("GT-N7000");
+  }

[xhwang, 2014/09/24 22:23:16]

Provide comments and link to bugs why we are doing this.

Also, personally I feel model != "GT-I9100" is more readable. But it could be
just me...

[qinmin, 2014/09/24 22:49:35]

Done.

+  return true;
 }
 
 // static