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

Unified Diff: webrtc/modules/video_coding/codecs/test/videoprocessor.cc

Issue 2993063002: Remove source file writer from VideoProcessor. (Closed)
Patch Set: video_quality_measurement updates. Created 3 years, 4 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: webrtc/modules/video_coding/codecs/test/videoprocessor.cc
diff --git a/webrtc/modules/video_coding/codecs/test/videoprocessor.cc b/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
index 5b2c4c081bb3b4eebe73bb0282f7409b2d546ff1..55303b82a248ab12f9aaeaae7dc696a491b09e54 100644
--- a/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
+++ b/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
@@ -112,7 +112,6 @@ VideoProcessor::VideoProcessor(webrtc::VideoEncoder* encoder,
PacketManipulator* packet_manipulator,
const TestConfig& config,
Stats* stats,
- FrameWriter* source_frame_writer,
IvfFileWriter* encoded_frame_writer,
FrameWriter* decoded_frame_writer)
: encoder_(encoder),
@@ -124,7 +123,6 @@ VideoProcessor::VideoProcessor(webrtc::VideoEncoder* encoder,
config_(config),
analysis_frame_reader_(analysis_frame_reader),
analysis_frame_writer_(analysis_frame_writer),
- source_frame_writer_(source_frame_writer),
encoded_frame_writer_(encoded_frame_writer),
decoded_frame_writer_(decoded_frame_writer),
initialized_(false),
@@ -210,16 +208,6 @@ bool VideoProcessor::ProcessFrame(int frame_number) {
return false;
}
- if (source_frame_writer_) {
- size_t length =
- CalcBufferSize(VideoType::kI420, buffer->width(), buffer->height());
- rtc::Buffer extracted_buffer(length);
- int extracted_length =
- ExtractBuffer(buffer, length, extracted_buffer.data());
- RTC_DCHECK_EQ(extracted_length, source_frame_writer_->FrameLength());
- RTC_CHECK(source_frame_writer_->WriteFrame(extracted_buffer.data()));
- }
-
uint32_t timestamp = FrameNumberToTimestamp(frame_number);
VideoFrame source_frame(buffer, timestamp, 0, webrtc::kVideoRotation_0);

Powered by Google App Engine
This is Rietveld 408576698