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

Unified Diff: content/renderer/media/webrtc/webrtc_video_frame_adapter.h

Issue 2456443002: Add callback to copy texture backed frames in WebRtcVideoFrameAdapter (Closed)
Patch Set: Created 4 years, 2 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: content/renderer/media/webrtc/webrtc_video_frame_adapter.h
diff --git a/content/renderer/media/webrtc/webrtc_video_frame_adapter.h b/content/renderer/media/webrtc/webrtc_video_frame_adapter.h
index 46dd87683d01edd03abe65e96c70bab912b21232..fadd5081650d40fe347c44f1556e0041b827ed0d 100644
--- a/content/renderer/media/webrtc/webrtc_video_frame_adapter.h
+++ b/content/renderer/media/webrtc/webrtc_video_frame_adapter.h
@@ -7,6 +7,7 @@
#include <stdint.h>
+#include "base/callback.h"
#include "media/base/video_frame.h"
#include "third_party/webrtc/common_video/include/video_frame_buffer.h"
@@ -17,8 +18,13 @@ namespace content {
// different threads, but that's safe since it's read-only.
class WebRtcVideoFrameAdapter : public webrtc::VideoFrameBuffer {
public:
- explicit WebRtcVideoFrameAdapter(
- const scoped_refptr<media::VideoFrame>& frame);
+ using CopyTextureFrameCallback =
+ base::Callback<void(const scoped_refptr<media::VideoFrame>&,
+ scoped_refptr<media::VideoFrame>*)>;
+
+ WebRtcVideoFrameAdapter(
+ const scoped_refptr<media::VideoFrame>& frame,
+ const CopyTextureFrameCallback& copy_texture_callback);
private:
int width() const override;
@@ -42,6 +48,7 @@ class WebRtcVideoFrameAdapter : public webrtc::VideoFrameBuffer {
~WebRtcVideoFrameAdapter() override;
scoped_refptr<media::VideoFrame> frame_;
mcasas 2016/10/26 16:06:02 nit: IINM this might be const as well
emircan 2016/10/27 00:37:56 I cant change it as it can be modified, see l.93 i
+ const CopyTextureFrameCallback copy_texture_callback_;
};
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698