Store SupportedCodecs in KeySystemInfo and KeySystems.

chrome/browser/media/encrypted_media_message_filter_android.cc

Index: chrome/browser/media/encrypted_media_message_filter_android.cc
diff --git a/chrome/browser/media/encrypted_media_message_filter_android.cc b/chrome/browser/media/encrypted_media_message_filter_android.cc
index c935fde54c6928dbecf022e5a54b07a33795be00..f1b9d46a65aafee810e3845c33b7dfaad22e2db0 100644
--- a/chrome/browser/media/encrypted_media_message_filter_android.cc
+++ b/chrome/browser/media/encrypted_media_message_filter_android.cc
@@ -25,26 +25,26 @@ enum CodecType {
 };
 
 struct CodecInfo {
-  android::SupportedCodecs codec;
+  media::SupportedCodecs codec;
   CodecType codec_type;
   const char* codec_name;
   const char* container_mime_type;
 };
 
 const CodecInfo kCodecsToQuery[] = {
-  {android::WEBM_VORBIS, CODEC_AUDIO, "vorbis", "video/webm"},
-  {android::WEBM_VP8, CODEC_VIDEO, "vp8", "video/webm"},
+  {media::WEBM_VORBIS, CODEC_AUDIO, "vorbis", "video/webm"},
+  {media::WEBM_VP8, CODEC_VIDEO, "vp8", "video/webm"},
 #if defined(USE_PROPRIETARY_CODECS)
-  {android::MP4_AAC, CODEC_AUDIO, "mp4a", "video/mp4"},
-  {android::MP4_AVC1, CODEC_VIDEO, "avc1", "video/mp4"}
+  {media::MP4_AAC, CODEC_AUDIO, "mp4a", "video/mp4"},
+  {media::MP4_AVC1, CODEC_VIDEO, "avc1", "video/mp4"}
 #endif  // defined(USE_PROPRIETARY_CODECS)
 };
 
-static android::SupportedCodecs GetSupportedCodecs(
+static media::SupportedCodecs GetSupportedCodecs(
     const SupportedKeySystemRequest& request,
     bool video_must_be_compositable) {
   const std::string& key_system = request.key_system;
-  android::SupportedCodecs supported_codecs = android::NO_CODECS;
+  media::SupportedCodecs supported_codecs = media::NO_CODECS;
 
   for (size_t i = 0; i < arraysize(kCodecsToQuery); ++i) {
     const CodecInfo& info = kCodecsToQuery[i];
@@ -56,8 +56,7 @@ static android::SupportedCodecs GetSupportedCodecs(
         MediaDrmBridge::IsKeySystemSupportedWithType(
             key_system, info.container_mime_type) &&
         MediaCodecBridge::CanDecode(info.codec_name, is_secure)) {
-      supported_codecs = static_cast<android::SupportedCodecs>(
-          supported_codecs | info.codec);
+      supported_codecs |= info.codec;
     }
   }
 
@@ -104,8 +103,7 @@ void EncryptedMediaMessageFilterAndroid::OnGetSupportedKeySystems(
   if (!MediaDrmBridge::IsKeySystemSupported(request.key_system))
     return;
 
-  DCHECK_EQ(request.codecs & android::INVALID_CODECS, android::NO_CODECS)
-      << "unrecognized codec";
+  DCHECK(!(request.codecs & media::INVALID_CODECS)) << "unrecognized codec";

[ddorwin, 2014/04/22 21:24:41]

DCHECK(request.codecs & media::ALL_CODECS) seems easier to read. Do we even need
INVALID_CODECS?

[xhwang, 2014/04/23 17:29:14]

Done.

   response->key_system = request.key_system;
   // TODO(qinmin): check composition is supported or not.
   response->compositing_codecs = GetSupportedCodecs(request, true);