Index: webrtc/video/send_statistics_proxy_unittest.cc |
diff --git a/webrtc/video/send_statistics_proxy_unittest.cc b/webrtc/video/send_statistics_proxy_unittest.cc |
index cb3bde442647856fab6cea7b41e575148105b45e..833057058389a53b8a11df7b87a5eb95bb029658 100644 |
--- a/webrtc/video/send_statistics_proxy_unittest.cc |
+++ b/webrtc/video/send_statistics_proxy_unittest.cc |
@@ -26,7 +26,6 @@ const uint32_t kSecondSsrc = 42; |
const uint32_t kFirstRtxSsrc = 18; |
const uint32_t kSecondRtxSsrc = 43; |
-const int kMinRequiredSamples = 200; |
const int kQpIdx0 = 21; |
const int kQpIdx1 = 39; |
} // namespace |
@@ -315,7 +314,7 @@ TEST_F(SendStatisticsProxyTest, SwitchContentTypeUpdatesHistograms) { |
const int kWidth = 640; |
const int kHeight = 480; |
- for (int i = 0; i < kMinRequiredSamples; ++i) |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) |
statistics_proxy_->OnIncomingFrame(kWidth, kHeight); |
// No switch, stats should not be updated. |
@@ -330,6 +329,25 @@ TEST_F(SendStatisticsProxyTest, SwitchContentTypeUpdatesHistograms) { |
EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.InputWidthInPixels")); |
} |
+TEST_F(SendStatisticsProxyTest, CpuLimitedResolutionUpdated) { |
+ const int kWidth = 640; |
+ const int kHeight = 480; |
+ |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) |
+ statistics_proxy_->OnIncomingFrame(kWidth, kHeight); |
+ |
+ statistics_proxy_->OnCpuRestrictedResolutionChanged(true); |
+ |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) |
+ statistics_proxy_->OnIncomingFrame(kWidth, kHeight); |
+ |
+ statistics_proxy_.reset(); |
+ EXPECT_EQ(1, |
+ metrics::NumSamples("WebRTC.Video.CpuLimitedResolutionInPercent")); |
+ EXPECT_EQ( |
+ 1, metrics::NumEvents("WebRTC.Video.CpuLimitedResolutionInPercent", 50)); |
+} |
+ |
TEST_F(SendStatisticsProxyTest, LifetimeHistogramIsUpdated) { |
const int64_t kTimeSec = 3; |
fake_clock_.AdvanceTimeMilliseconds(kTimeSec * 1000); |
@@ -350,7 +368,7 @@ TEST_F(SendStatisticsProxyTest, VerifyQpHistogramStats_Vp8) { |
CodecSpecificInfo codec_info; |
codec_info.codecType = kVideoCodecVP8; |
- for (int i = 0; i < kMinRequiredSamples; ++i) { |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) { |
codec_info.codecSpecific.VP8.simulcastIdx = 0; |
encoded_image.qp_ = kQpIdx0; |
statistics_proxy_->OnSendEncodedImage(encoded_image, &codec_info); |
@@ -375,7 +393,7 @@ TEST_F(SendStatisticsProxyTest, VerifyQpHistogramStats_Vp8OneSsrc) { |
CodecSpecificInfo codec_info; |
codec_info.codecType = kVideoCodecVP8; |
- for (int i = 0; i < kMinRequiredSamples; ++i) { |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) { |
codec_info.codecSpecific.VP8.simulcastIdx = 0; |
encoded_image.qp_ = kQpIdx0; |
statistics_proxy_->OnSendEncodedImage(encoded_image, &codec_info); |
@@ -391,7 +409,7 @@ TEST_F(SendStatisticsProxyTest, VerifyQpHistogramStats_Vp9) { |
codec_info.codecType = kVideoCodecVP9; |
codec_info.codecSpecific.VP9.num_spatial_layers = 2; |
- for (int i = 0; i < kMinRequiredSamples; ++i) { |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) { |
encoded_image.qp_ = kQpIdx0; |
codec_info.codecSpecific.VP9.spatial_idx = 0; |
statistics_proxy_->OnSendEncodedImage(encoded_image, &codec_info); |
@@ -417,7 +435,7 @@ TEST_F(SendStatisticsProxyTest, VerifyQpHistogramStats_Vp9OneSpatialLayer) { |
codec_info.codecType = kVideoCodecVP9; |
codec_info.codecSpecific.VP9.num_spatial_layers = 1; |
- for (int i = 0; i < kMinRequiredSamples; ++i) { |
+ for (int i = 0; i < SendStatisticsProxy::kMinRequiredMetricsSamples; ++i) { |
encoded_image.qp_ = kQpIdx0; |
codec_info.codecSpecific.VP9.spatial_idx = 0; |
statistics_proxy_->OnSendEncodedImage(encoded_image, &codec_info); |