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

Unified Diff: ppapi/proxy/video_encoder_resource.h

Issue 905023005: Pepper: PPB_VideoEncoder implementation (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Update after bbudge's review Created 5 years, 10 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: ppapi/proxy/video_encoder_resource.h
diff --git a/ppapi/proxy/video_encoder_resource.h b/ppapi/proxy/video_encoder_resource.h
index 585d5c14576585f15e90ac012a461e7aca62d7e0..ce783a08af24f29c900377881bee6eba9dcff68a 100644
--- a/ppapi/proxy/video_encoder_resource.h
+++ b/ppapi/proxy/video_encoder_resource.h
@@ -5,21 +5,34 @@
#ifndef PPAPI_PROXY_VIDEO_ENCODER_RESOURCE_H_
#define PPAPI_PROXY_VIDEO_ENCODER_RESOURCE_H_
+#include <deque>
+
#include "base/memory/ref_counted.h"
+#include "base/memory/scoped_ptr.h"
+#include "base/memory/scoped_vector.h"
#include "ppapi/proxy/connection.h"
#include "ppapi/proxy/plugin_resource.h"
+#include "ppapi/shared_impl/media_stream_buffer_manager.h"
#include "ppapi/shared_impl/resource.h"
#include "ppapi/thunk/ppb_video_encoder_api.h"
+namespace base {
+class SharedMemory;
+}
+
namespace ppapi {
class TrackedCallback;
namespace proxy {
+class SerializedHandle;
+class VideoFrameResource;
+
class PPAPI_PROXY_EXPORT VideoEncoderResource
: public PluginResource,
- public thunk::PPB_VideoEncoder_API {
+ public thunk::PPB_VideoEncoder_API,
+ public ppapi::MediaStreamBufferManager::Delegate {
public:
VideoEncoderResource(Connection connection, PP_Instance instance);
~VideoEncoderResource() override;
@@ -27,6 +40,28 @@ class PPAPI_PROXY_EXPORT VideoEncoderResource
thunk::PPB_VideoEncoder_API* AsPPB_VideoEncoder_API() override;
private:
+ struct ShmBuffer {
+ ShmBuffer(base::SharedMemoryHandle handle, uint32_t id, uint32_t size);
+ ~ShmBuffer();
+
+ scoped_ptr<base::SharedMemory> shm;
+ // Index of the buffer in the ScopedVector. Buffers have the id in
+ // the plugin and the host.
+ uint32_t id;
+ uint32_t size;
+ };
+
+ struct BitstreamBuffer {
+ BitstreamBuffer(uint32_t id, uint32_t size, bool key_frame);
+ BitstreamBuffer(const BitstreamBuffer& other);
+ ~BitstreamBuffer();
+
+ // Index of the buffer in the ScopedVector. Same as ShmBuffer::id.
+ uint32_t id;
+ uint32_t size;
+ bool key_frame;
+ };
+
// PPB_VideoEncoder_API implementation.
int32_t GetSupportedProfiles(
const PP_ArrayOutput& output,
@@ -53,6 +88,71 @@ class PPAPI_PROXY_EXPORT VideoEncoderResource
uint32_t framerate) override;
void Close() override;
+ // PluginResource implementation.
+ void OnReplyReceived(const ResourceMessageReplyParams& params,
+ const IPC::Message& msg) override;
+
+ // Reply message handlers for operations that are done in the host.
+ void OnPluginMsgGetSupportedProfilesReply(
+ const PP_ArrayOutput& output,
+ const scoped_refptr<TrackedCallback>& callback,
+ const ResourceMessageReplyParams& params,
+ const std::vector<PP_VideoProfileDescription>& profiles);
+ void OnPluginMsgInitializeReply(const ResourceMessageReplyParams& params,
+ uint32_t buffer_length,
+ uint32_t input_frame_count,
+ const PP_Size& input_coded_size);
+ void OnPluginMsgGetVideoFramesReply(const ResourceMessageReplyParams& params,
+ uint32_t frame_count,
+ uint32_t frame_length,
+ const PP_Size& frame_size);
+ void OnPluginMsgEncodeReply(const scoped_refptr<TrackedCallback>& callback,
+ const ResourceMessageReplyParams& params,
+ uint32_t frame_id);
+
+ // Unsolicited reply message handlers.
+ void OnPluginMsgBitstreamBufferReady(const ResourceMessageReplyParams& params,
+ uint32_t buffer_id,
+ uint32_t buffer_size,
+ bool key_frame);
+ void OnPluginMsgNotifyError(const ResourceMessageReplyParams& params,
+ int32_t error);
+
+ // Internal utility functions.
+ void NotifyError(int32_t error);
+ void NotifyGetVideoFrameCallbacks();
+ void NotifyGetVideoFrameCallbacksError(int32_t error);
+ void WriteBitstreamBuffer(PP_BitstreamBuffer* bitstream_buffer,
+ const BitstreamBuffer& buffer);
+ void ReleaseFrames();
+
+ bool initialized_;
+ int32_t encoder_last_error_;
+
+ int32_t input_frame_count_;
+ PP_Size input_coded_size_;
+
+ bool waiting_for_video_frames_;
+
+ MediaStreamBufferManager buffer_manager_;
+
+ typedef std::map<PP_Resource, scoped_refptr<VideoFrameResource> >
+ VideoFrameMap;
+ VideoFrameMap video_frames_;
+
+ std::deque<std::pair<PP_Resource*, scoped_refptr<TrackedCallback> > >
+ pending_get_video_frame_calls_;
+
+ ScopedVector<ShmBuffer> shm_buffers_;
+
+ std::deque<BitstreamBuffer> available_bitstream_buffers_;
+ typedef std::map<void*, uint32_t> BitstreamBufferMap;
+ BitstreamBufferMap bitstream_buffers_map_;
bbudge 2015/02/10 22:47:39 nit: s/bitstream_buffers_map_/bitstream_buffer_map
llandwerlin-old 2015/02/11 18:42:25 Done.
+
+ scoped_refptr<TrackedCallback> initialize_callback_;
+ scoped_refptr<TrackedCallback> get_bitstream_buffer_callback_;
+ PP_BitstreamBuffer* get_bitstream_buffer_data_;
+
DISALLOW_COPY_AND_ASSIGN(VideoEncoderResource);
};

Powered by Google App Engine
This is Rietveld 408576698