Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(255)

Unified Diff: media/base/video_decoder_config.h

Issue 418193003: Using PROFILE_ANY for vp8 and vp9 (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/base/video_decoder_config.h
diff --git a/media/base/video_decoder_config.h b/media/base/video_decoder_config.h
index 3cc33d927158a5c279c2e91b0e5dbd9aad6467e1..475cb09f2bf408cb561c9aedb688eb2da931c712 100644
--- a/media/base/video_decoder_config.h
+++ b/media/base/video_decoder_config.h
@@ -58,10 +58,12 @@ enum VideoCodecProfile {
H264PROFILE_MULTIVIEWHIGH = 10,
H264PROFILE_MAX = H264PROFILE_MULTIVIEWHIGH,
VP8PROFILE_MIN = 11,
- VP8PROFILE_MAIN = VP8PROFILE_MIN,
+ VP8PROFILE_NOT_NEEDED = VP8PROFILE_MIN,
+ VP8PROFILE_MAIN = VP8PROFILE_NOT_NEEDED,
VP8PROFILE_MAX = VP8PROFILE_MAIN,
VP9PROFILE_MIN = 12,
xhwang 2014/07/30 21:21:41 We don't need VP8PROFILE_MAIN, VP8PROFILE_MAX and
amogh.bihani 2014/07/31 05:07:40 Done.
xhwang 2014/07/31 17:12:44 Can we drop VP8PROFILE_MIN and VP8PROFILE_MAX also
- VP9PROFILE_MAIN = VP9PROFILE_MIN,
+ VP9PROFILE_NOT_NEEDED = VP9PROFILE_MIN,
+ VP9PROFILE_MAIN = VP9PROFILE_NOT_NEEDED,
VP9PROFILE_MAX = VP9PROFILE_MAIN,
xhwang 2014/07/30 21:21:41 ditto
amogh.bihani 2014/07/31 05:07:40 Done.
xhwang 2014/07/31 17:12:44 ditto
VIDEO_CODEC_PROFILE_MAX = VP9PROFILE_MAX,
};

Powered by Google App Engine
This is Rietveld 408576698