Fix wrong selection of VP8 video decoder on some Samsung devices.

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..77a52b8f54e9922277269074eae46eae62672992 100644
--- a/media/base/android/media_codec_bridge.cc
+++ b/media/base/android/media_codec_bridge.cc
@@ -175,9 +175,13 @@ bool MediaCodecBridge::IsKnownUnaccelerated(const std::string& mime_type,
         codecs_info[i].direction == direction) {
       // It would be nice if MediaCodecInfo externalized some notion of
       // HW-acceleration but it doesn't. Android Media guidance is that the
-      // prefix below is always used for SW decoders, so that's what we use.
-      if (!StartsWithASCII(codecs_info[i].name, "OMX.google.", true))
-        return false;
+      // "OMX.google" prefix is always used for SW decoders, so that's what we
+      // use. "OMX.SEC.vp8.dec" codec don't work when trying to decode to a
+      // surface, so report it as SW decoder as well.

[qinmin, 2014/09/26 05:00:28]

the comments is wierd. This function is checking whether the decoder is HW
accelerated or not, it has nothing to do with whether it can decode to a
surface.
The only place that we call this function is android_video_decode_accelerator.cc
and android_video_encode_accelerator.cc

[AlexGlaznev, 2014/09/26 18:01:37]

Done.

+      DVLOG(1) << __PRETTY_FUNCTION__ << "Default codec for " << mime_type <<
+          ": " << codecs_info[i].name;
+      return (StartsWithASCII(codecs_info[i].name, "OMX.google.", true) ||
+          StartsWithASCII(codecs_info[i].name, "OMX.SEC.", true));
     }
   }
   return true;