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

Unified Diff: remoting/host/video_frame_recorder.cc

Issue 1846893002: Interface with webrtc through encoded frames (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed sergeyu comments Created 4 years, 8 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: remoting/host/video_frame_recorder.cc
diff --git a/remoting/host/video_frame_recorder.cc b/remoting/host/video_frame_recorder.cc
index 3cec5b7c7c3867e41d7b4de503cdb06937c1f075..0c15601c5e51fcd750a94e2499cf37f413d15271 100644
--- a/remoting/host/video_frame_recorder.cc
+++ b/remoting/host/video_frame_recorder.cc
@@ -43,7 +43,8 @@ class VideoFrameRecorder::RecordingVideoEncoder : public VideoEncoder {
// remoting::VideoEncoder interface.
void SetLosslessEncode(bool want_lossless) override;
void SetLosslessColor(bool want_lossless) override;
- scoped_ptr<VideoPacket> Encode(const webrtc::DesktopFrame& frame) override;
+ scoped_ptr<VideoPacket> Encode(const webrtc::DesktopFrame& frame,
+ uint32_t flags = 0) override;
private:
scoped_ptr<VideoEncoder> encoder_;
@@ -87,7 +88,8 @@ void VideoFrameRecorder::RecordingVideoEncoder::SetLosslessColor(
}
scoped_ptr<VideoPacket> VideoFrameRecorder::RecordingVideoEncoder::Encode(
- const webrtc::DesktopFrame& frame) {
+ const webrtc::DesktopFrame& frame,
+ uint32_t flags) {
// If this is the first Encode() then store the TaskRunner and inform the
// VideoFrameRecorder so it can post set_enable_recording() on it.
if (!encoder_task_runner_.get()) {
@@ -115,7 +117,7 @@ scoped_ptr<VideoPacket> VideoFrameRecorder::RecordingVideoEncoder::Encode(
base::Passed(&frame_copy)));
}
- return encoder_->Encode(frame);
+ return encoder_->Encode(frame, flags);
}
VideoFrameRecorder::VideoFrameRecorder()

Powered by Google App Engine
This is Rietveld 408576698