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

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

Issue 2707023008: Step #2: Add batch mode to VideoProcessor integration tests. (Closed)
Patch Set: Created 3 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: 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 17a13c3119fa56f56ddd170921c3e68eec5bc3de..f65b885683ce34c5882a567cf312625eb0203cd5 100644
--- a/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
+++ b/webrtc/modules/video_coding/codecs/test/videoprocessor.cc
@@ -201,16 +201,14 @@ void VideoProcessorImpl::SetRates(int bit_rate, int frame_rate) {
num_spatial_resizes_ = 0;
}
-// TODO(brandtr): Update implementation of EncodedFrameSize and EncodedFrameType
-// to support batch processing in the caller.
-size_t VideoProcessorImpl::EncodedFrameSize() {
- RTC_CHECK(!frame_infos_.empty());
- return frame_infos_.back().encoded_frame_size;
+size_t VideoProcessorImpl::EncodedFrameSize(int frame_number) {
+ RTC_CHECK_LT(frame_number, frame_infos_.size());
+ return frame_infos_[frame_number].encoded_frame_size;
}
-FrameType VideoProcessorImpl::EncodedFrameType() {
- RTC_CHECK(!frame_infos_.empty());
- return frame_infos_.back().encoded_frame_type;
+FrameType VideoProcessorImpl::EncodedFrameType(int frame_number) {
+ RTC_CHECK_LT(frame_number, frame_infos_.size());
+ return frame_infos_[frame_number].encoded_frame_type;
}
int VideoProcessorImpl::NumberDroppedFrames() {

Powered by Google App Engine
This is Rietveld 408576698