Using PROFILE_ANY for vp8 and vp9

content/common/gpu/media/android_video_decode_accelerator.cc

Index: content/common/gpu/media/android_video_decode_accelerator.cc
diff --git a/content/common/gpu/media/android_video_decode_accelerator.cc b/content/common/gpu/media/android_video_decode_accelerator.cc
index f72e1b257d77c8bc9f5c243ab2f1e34d5faad042..5bf0fbba4364880342929ab393e35b58bc8762fc 100644
--- a/content/common/gpu/media/android_video_decode_accelerator.cc
+++ b/content/common/gpu/media/android_video_decode_accelerator.cc
@@ -89,7 +89,7 @@ bool AndroidVideoDecodeAccelerator::Initialize(media::VideoCodecProfile profile,
 
   client_ = client;
 
-  if (profile == media::VP8PROFILE_MAIN) {
+  if (profile == media::VP8PROFILE_UNSPECIFIED) {

[xhwang, 2014/07/31 17:12:44]

I think here you mean for "vp8", "profile" is "unspecified" in the spec. But
when I look at this, "unspecified" means that it should be specified but not
specified, which isn't the case here. So "unspecified" here is ambiguous.

I think I still like "NOT_NEEDED" better. Pawel, WDYT?

Another option is to use "NOT_APPLICABLE", as in N/A:
http://en.wikipedia.org/wiki/N/a

Based on wikipedia, the meaning is exact what Pawel mentioned in his comment.

Also, we only need one "NOT_NEEDED", which can be shared by all codecs that
don't need a profile (e.g. vp8/vp9). So how about just having:
media::PROFILE_NOT_NEEDED, or
media::PROFILE_NOT_APPLICABLE

[amogh.bihani, 2014/08/01 05:16:24]

Na has a wiki page! :o
Won't creating single NOT_APPLICABLE profile for vp8/9 will make us lose the
distinction that we have between these two codecs, which might be used perhaps
later.

     codec_ = media::kCodecVP8;
   } else {
     // TODO(dwkang): enable H264 once b/8125974 is fixed.