Fix mime type mappings when the unified media pipeline is enabled.

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 7650ecd7f7813a11352163f4f099d073ddef58a6..97fcc654f20a290bfbdbab5aecdf3cd565e1d63e 100644
--- a/media/base/android/media_codec_util.cc
+++ b/media/base/android/media_codec_util.cc
@@ -245,4 +245,14 @@ bool MediaCodecUtil::RegisterMediaCodecUtil(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
+// static
+bool MediaCodecUtil::IsVp8Blacklisted() {
+  if (!IsMediaCodecAvailable())

[ddorwin, 2016/02/19 19:53:23]

This probably isn't relevant anymore. Don't call this if you don't have
MediaCodec. You could DCHECK instead.

[DaleCurtis, 2016/02/19 20:28:35]

Perhaps, but the rest of the methods in the file do this, so I'd like to keep it
for consistency. It also simplifies some of the checks elsewhere (e.g.
StreamParserFactory).

[ddorwin, 2016/02/19 20:47:00]

It makes more sense now that we're not asking about the blacklist.

+    return false;
+
+  JNIEnv* env = AttachCurrentThread();
+  ScopedJavaLocalRef<jstring> j_mime = ConvertUTF8ToJavaString(env, "vp8");
+  return Java_MediaCodecUtil_isDecoderSupportedForDevice(env, j_mime.obj());

[ddorwin, 2016/02/19 19:53:23]

Invert.

[DaleCurtis, 2016/02/19 20:28:35]

Done.

+}
+
 }  // namespace media