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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: ppapi/api/dev/pp_video_dev.idl

Issue 1677133003: Implemented parsing of HEVC codec ids (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@parse-codec-id
Patch Set: Updated comments about VideoCodecProfile enum Created 4 years, 10 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
« media/mojo/interfaces/media_types.mojom ('K') | « media/mojo/services/media_type_converters.cc ('k') | ppapi/c/dev/pp_video_dev.h » ('j') | ppapi/c/dev/pp_video_dev.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/api/dev/pp_video_dev.idl
diff --git a/ppapi/api/dev/pp_video_dev.idl b/ppapi/api/dev/pp_video_dev.idl
index 1c70342effcf9a75ebca77bc56ce6d6dbce7ce19..65c3faff0164f69911d05f5292909afaf980498b 100644
--- a/ppapi/api/dev/pp_video_dev.idl
+++ b/ppapi/api/dev/pp_video_dev.idl
@@ -14,7 +14,8 @@
* Keep the values in this enum unique, as they imply format (h.264 vs. VP8,
* for example), and keep the values for a particular format grouped together
* for clarity.
- * Note: Keep these in sync with media::VideoCodecProfile.
+ * Note: These are translated into media::VideoCodecProfile in
+ * content/renderer/pepper/ppb_video_decoder_impl.cc, keep them in sync.
*/
[assert_size(4)]
enum PP_VideoDecoder_Profile {
@@ -31,7 +32,11 @@ enum PP_VideoDecoder_Profile {
PP_VIDEODECODER_H264PROFILE_SCALABLEHIGH = 9,
PP_VIDEODECODER_H264PROFILE_STEREOHIGH = 10,
PP_VIDEODECODER_H264PROFILE_MULTIVIEWHIGH = 11,
- PP_VIDEODECODER_VP8PROFILE_ANY = 12
+ PP_VIDEODECODER_VP8PROFILE_ANY = 12,
+ PP_VIDEODECODER_VP9PROFILE_ANY = 13,
+ PP_VIDEODECODER_HEVCPROFILE_MAIN = 14,
ddorwin 2016/02/26 18:54:17 Does it matter that we have no plans to support ce
servolk 2016/02/26 19:50:19 I think that doesn't matter, as that can easily ch
+ PP_VIDEODECODER_HEVCPROFILE_MAIN10 = 15,
+ PP_VIDEODECODER_HEVCPROFILE_MAIN_STILL_PICTURE = 16
};
/**
« media/mojo/interfaces/media_types.mojom ('K') | « media/mojo/services/media_type_converters.cc ('k') | ppapi/c/dev/pp_video_dev.h » ('j') | ppapi/c/dev/pp_video_dev.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698