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

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

Issue 2999613002: Minor improvements to VideoProcessorIntegrationTest. (Closed)
Patch Set: Fix iOS more. 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
« no previous file with comments | « no previous file | webrtc/modules/video_coding/codecs/test/videoprocessor.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/modules/video_coding/codecs/test/plot_videoprocessor_integrationtest.cc
diff --git a/webrtc/modules/video_coding/codecs/test/plot_videoprocessor_integrationtest.cc b/webrtc/modules/video_coding/codecs/test/plot_videoprocessor_integrationtest.cc
index 56d8955dc148060bf57e2a016597f93763eb7719..b856c9c11b1bcd997b5906a124b87af30b1c3d8f 100644
--- a/webrtc/modules/video_coding/codecs/test/plot_videoprocessor_integrationtest.cc
+++ b/webrtc/modules/video_coding/codecs/test/plot_videoprocessor_integrationtest.cc
@@ -40,6 +40,8 @@ const VisualizationParams kVisualizationParams = {
};
const bool kVerboseLogging = true;
+
+const int kNumFrames = 299;
} // namespace
// Tests for plotting statistics from logs.
@@ -62,8 +64,8 @@ class PlotVideoProcessorIntegrationTest
0, // update_index
bitrate_, framerate_,
0); // frame_index_rate_update
- rate_profile.frame_index_rate_update[1] = kNumFramesLong + 1;
- rate_profile.num_frames = kNumFramesLong;
+ rate_profile.frame_index_rate_update[1] = kNumFrames + 1;
+ rate_profile.num_frames = kNumFrames;
// Codec/network settings.
SetProcessParams(&config_, kHwCodec, kUseSingleCore, kPacketLoss,
@@ -81,14 +83,14 @@ class PlotVideoProcessorIntegrationTest
// clang-format off
SetRateControlThresholds(
rc_thresholds,
- 0, // update_index
- kNumFramesLong + 1, // max_num_dropped_frames
- 10000, // max_key_frame_size_mismatch
- 10000, // max_delta_frame_size_mismatch
- 10000, // max_encoding_rate_mismatch
- kNumFramesLong + 1, // max_time_hit_target
- 0, // num_spatial_resizes
- 1); // num_key_frames
+ 0, // update_index
+ kNumFrames + 1, // max_num_dropped_frames
+ 10000, // max_key_frame_size_mismatch
+ 10000, // max_delta_frame_size_mismatch
+ 10000, // max_encoding_rate_mismatch
+ kNumFrames + 1, // max_time_hit_target
+ 0, // num_spatial_resizes
+ 1); // num_key_frames
// clang-format on
ProcessFramesAndVerify(quality_thresholds, rate_profile, rc_thresholds,
« no previous file with comments | « no previous file | webrtc/modules/video_coding/codecs/test/videoprocessor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698