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

Unified Diff: media/mojo/services/mojo_video_decoder_service.h

Issue 2640153004: Add mailbox-based Mojo VideoFrame variant. (Closed)
Patch Set: Created 3 years, 11 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/mojo/services/mojo_video_decoder_service.h
diff --git a/media/mojo/services/mojo_video_decoder_service.h b/media/mojo/services/mojo_video_decoder_service.h
index ce824eee56c76b9293fb7244705deb671068eb4f..c0e0b45e6d45fb08e0e3c505696ef8270fcaa662 100644
--- a/media/mojo/services/mojo_video_decoder_service.h
+++ b/media/mojo/services/mojo_video_decoder_service.h
@@ -9,9 +9,14 @@
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
+#include "base/optional.h"
#include "media/base/decode_status.h"
#include "media/mojo/interfaces/video_decoder.mojom.h"
+namespace gpu {
+struct SyncToken;
+};
+
namespace media {
class MojoDecoderBufferReader;
@@ -49,6 +54,8 @@ class MojoVideoDecoderService : public mojom::VideoDecoder {
void OnDecoderReset(const ResetCallback& callback);
void OnDecoderOutput(const scoped_refptr<VideoFrame>& frame);
+ void OnVideoFrameDestroyed(
xhwang 2017/01/25 18:25:09 nit: In VideoFrame we have DestructionObserver, ho
sandersd (OOO until July 31) 2017/01/26 23:49:05 Renamed to OnReleaseMailbox().
+ const base::Optional<gpu::SyncToken>& release_sync_token);
mojom::VideoDecoderClientAssociatedPtr client_;
std::unique_ptr<MojoDecoderBufferReader> mojo_decoder_buffer_reader_;

Powered by Google App Engine
This is Rietveld 408576698