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

media/base/android/media_codec_util.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "media/base/android/media_codec_util.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 
@@ skipping 227 matching lines @@
     return true;
 
   return (spec.find("m3u8") != std::string::npos);
 }
 
 // static
 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