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

Unified Diff: media/video/video_encode_accelerator.h

Issue 185403020: Make VEA client of command buffer; move sync. IPC to VDA/VEA::Initialize() (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: f2a9ccb5 Rebase, posciak@ comments. Created 6 years, 9 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/video/video_encode_accelerator.h
diff --git a/media/video/video_encode_accelerator.h b/media/video/video_encode_accelerator.h
index 169d124486ff100e5b8b2fe04cb4175b088df5f7..00bd180267ff7665d51b0923c2b755e35ad4f129 100644
--- a/media/video/video_encode_accelerator.h
+++ b/media/video/video_encode_accelerator.h
@@ -51,9 +51,6 @@ class MEDIA_EXPORT VideoEncodeAccelerator {
// Interface for clients that use VideoEncodeAccelerator.
class MEDIA_EXPORT Client {
public:
- // Callback to notify client that encoder has been successfully initialized.
- virtual void NotifyInitializeDone() = 0;
-
// Callback to tell the client what size of frames and buffers to provide
// for input and output. The VEA disclaims use or ownership of all
// previously provided buffers once this callback is made.
@@ -94,8 +91,9 @@ class MEDIA_EXPORT VideoEncodeAccelerator {
// Video encoder functions.
- // Initialize the video encoder with a specific configuration. Called once
- // per encoder construction.
+ // Initializes the video encoder with specific configuration. Called once per
+ // encoder construction. This call is synchronous and returns true iff
+ // initialization is successful.
// Parameters:
// |input_format| is the frame format of the input stream (as would be
// reported by VideoFrame::format() for frames passed to Encode()).
@@ -108,7 +106,7 @@ class MEDIA_EXPORT VideoEncodeAccelerator {
// |client| is the client of this video encoder. The provided pointer must
// be valid until Destroy() is called.
// TODO(sheu): handle resolution changes. http://crbug.com/249944
- virtual void Initialize(VideoFrame::Format input_format,
+ virtual bool Initialize(VideoFrame::Format input_format,
const gfx::Size& input_visible_size,
VideoCodecProfile output_profile,
uint32 initial_bitrate,
« media/video/video_decode_accelerator.h ('K') | « media/video/video_decode_accelerator.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698