| Index: webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
|
| diff --git a/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h b/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
|
| index 60058eea25360cfc6fdb472c80827aba7679f604..d5471d90edbd0328153474006ee5f8c7eab516f8 100644
|
| --- a/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
|
| +++ b/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h
|
| @@ -500,9 +500,7 @@ class VideoProcessorIntegrationTest : public testing::Test {
|
| // In batch mode, we calculate the metrics for all frames after all frames
|
| // have been sent for encoding.
|
|
|
| - // TODO(brandtr): Refactor "frame number accounting" so we don't have to
|
| - // call ProcessFrame num_frames+1 times here.
|
| - for (frame_number = 0; frame_number <= num_frames; ++frame_number) {
|
| + for (frame_number = 0; frame_number < num_frames; ++frame_number) {
|
| processor_->ProcessFrame(frame_number);
|
| }
|
|
|
| @@ -547,16 +545,13 @@ class VideoProcessorIntegrationTest : public testing::Test {
|
| processor_->SetRates(bitrate_kbps_, framerate_);
|
| }
|
| }
|
| - // TODO(brandtr): Refactor "frame number accounting" so we don't have to
|
| - // call ProcessFrame one extra time here.
|
| - processor_->ProcessFrame(frame_number);
|
| }
|
|
|
| // Verify rate control metrics for all frames (if in batch mode), or for all
|
| // frames since the last rate update (if not in batch mode).
|
| PrintAndMaybeVerifyRateControlMetrics(update_index, rc_thresholds);
|
| EXPECT_EQ(num_frames, frame_number);
|
| - EXPECT_EQ(num_frames + 1, static_cast<int>(stats_.stats_.size()));
|
| + EXPECT_EQ(num_frames, static_cast<int>(stats_.stats_.size()));
|
|
|
| // Release encoder and decoder to make sure they have finished processing.
|
| processor_->Release();
|
|
|