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

Unified Diff: media/mojo/interfaces/media_types.mojom

Issue 2933573003: media: Followup changes to VideoFrame StructTraits. (Closed)
Patch Set: const& Created 3 years, 6 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
« no previous file with comments | « media/mojo/common/media_type_converters.cc ('k') | media/mojo/interfaces/video_frame_struct_traits.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/interfaces/media_types.mojom
diff --git a/media/mojo/interfaces/media_types.mojom b/media/mojo/interfaces/media_types.mojom
index e206edbd6d082a8b73832769d2be48655f1a7e9d..5448756c7c511e13472f9804ac0120240b7a874e 100644
--- a/media/mojo/interfaces/media_types.mojom
+++ b/media/mojo/interfaces/media_types.mojom
@@ -213,22 +213,24 @@ struct VideoFrame {
// Natural size of the frame.
gfx.mojom.Size natural_size;
- // True if end of stream.
- bool end_of_stream;
-
// Timestamp in microseconds of the associated frame.
mojo.common.mojom.TimeDelta timestamp;
- // Contents of the video frame. Will be null for end of stream frames.
- VideoFrameData? data;
+ // Contents of the video frame (or EOS marker).
+ VideoFrameData data;
};
// Possible choices for storing VideoFrame data.
union VideoFrameData {
+ EosVideoFrameData eos_data;
SharedBufferVideoFrameData shared_buffer_data;
MailboxVideoFrameData mailbox_data;
};
+// A marker for EOS frames.
+struct EosVideoFrameData {
+};
+
// This defines video frame data stored in a Mojo shared buffer.
struct SharedBufferVideoFrameData {
// Reference to the shared memory containing the frame's data.
« no previous file with comments | « media/mojo/common/media_type_converters.cc ('k') | media/mojo/interfaces/video_frame_struct_traits.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698