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

Unified Diff: chrome/gpu/arc_video_accelerator.h

Issue 1945543002: Use explicit flush for ArcVideoAccelerator. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: address posciak's comments Created 4 years, 7 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: chrome/gpu/arc_video_accelerator.h
diff --git a/chrome/gpu/arc_video_accelerator.h b/chrome/gpu/arc_video_accelerator.h
index bb7fa622358e9f37b7a4309f4386f0be2f2f23c6..bb731704c6a4ef636b4d1596d2cb705a6c6b5c46 100644
--- a/chrome/gpu/arc_video_accelerator.h
+++ b/chrome/gpu/arc_video_accelerator.h
@@ -27,13 +27,8 @@ enum PortType {
PORT_COUNT = 2,
};
-enum BufferFlag {
- BUFFER_FLAG_EOS = 1 << 0,
-};
-
struct BufferMetadata {
int64_t timestamp = 0; // in microseconds
- uint32_t flags = 0; // Flags defined in BufferFlag.
uint32_t bytes_used = 0;
};
@@ -101,6 +96,9 @@ class ArcVideoAccelerator {
// Called as a completion notification for Reset().
virtual void OnResetDone() = 0;
+
+ // Called as a completion notification for Flush().
+ virtual void OnFlushDone() = 0;
};
// Initializes the ArcVideoAccelerator with specific configuration. This
@@ -142,6 +140,11 @@ class ArcVideoAccelerator {
// and there won't be more callbacks.
virtual void Reset() = 0;
+ // Flushes the accelerator. After all the output buffers pending decode have
+ // been returned to client by OnBufferDone(), Client::OnFlushDone() will be
+ // called.
+ virtual void Flush() = 0;
+
virtual ~ArcVideoAccelerator() {}
};

Powered by Google App Engine
This is Rietveld 408576698