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

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

Issue 2951233002: media: Implement MojoVideoDecoder::CanReadWithoutStalling(). (Closed)
Patch Set: 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/clients/mojo_video_decoder.cc ('k') | media/mojo/services/mojo_video_decoder_service.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/mojo/interfaces/video_decoder.mojom
diff --git a/media/mojo/interfaces/video_decoder.mojom b/media/mojo/interfaces/video_decoder.mojom
index cade79cb74b5e3ff8bee7d55f6ba83a384bca54c..36ae91fd26430098d7d4567019679efd14d5d35e 100644
--- a/media/mojo/interfaces/video_decoder.mojom
+++ b/media/mojo/interfaces/video_decoder.mojom
@@ -75,9 +75,16 @@ interface VideoDecoder {
};
interface VideoDecoderClient {
- // Output a decoded frame. Frames must be output in presentation order. If
- // |release_token| is provided, the client should call OnReleaseMailbox() when
- // it is finished with the frame.
+ // Output a decoded frame. Frames must be output in presentation order.
+ //
+ // When |can_read_without_stalling| is false, preroll is disabled. This is
+ // necessary if the decoder cannot guarantee that it can output another frame,
+ // for example if output buffers are limited or configuration changes require
+ // the return of all outstanding frames.
+ //
+ // If |release_token| is provided, the client shall call OnReleaseMailbox()
+ // when it is finished using the frame.
OnVideoFrameDecoded(VideoFrame frame,
+ bool can_read_without_stalling,
mojo.common.mojom.UnguessableToken? release_token);
};
« no previous file with comments | « media/mojo/clients/mojo_video_decoder.cc ('k') | media/mojo/services/mojo_video_decoder_service.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698