commit e0da9eab5f88cd3b427f94ddb1773ea0f90bf341
author Ilya Nikolaevskiy <ilnik@webrtc.org> Wed Nov 08 14:39:02 2017 +0100
committer Commit Bot <commit-bot@chromium.org> Fri Nov 10 15:11:25 2017 +0000
tree a8c11574a9001114c7158fcbe9bd5854dcca8588
parent fb65b10fa1cd034e2f2e3236cacda510959b2b7c

Don't count frames passed to VP8 BWE disabled streams as being encoded

This excludes these frames from being counted as dropped by encoder.
Also fix bitrate projected distribution for vp9 svc for outliers
detection.

Bug: webrtc:8497
Change-Id: Id37487456170c61e2323a660668f0c319ea5831d
Reviewed-on: https://webrtc-review.googlesource.com/21223
Commit-Queue: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#20632}

modules/video_coding/generic_encoder_unittest.cc

diff --git a/modules/video_coding/generic_encoder_unittest.cc b/modules/video_coding/generic_encoder_unittest.cc
index caada6e..48c152f 100644
--- a/modules/video_coding/generic_encoder_unittest.cc
+++ b/modules/video_coding/generic_encoder_unittest.cc
@@ -185,6 +185,7 @@
   VideoCodec::TimingFrameTriggerThresholds thresholds;
   thresholds.delay_ms = 1;  // Make all frames timing frames.
   callback.SetTimingFramesThresholds(thresholds);
+  callback.OnTargetBitrateChanged(500, 0);
 
   // Verify a single frame works with encode start time set.
   callback.OnEncodeStarted(timestamp, 0);
@@ -208,6 +209,8 @@
   codec_specific.codecSpecific.generic.simulcast_idx = 0;
   FakeEncodedImageCallback sink;
   VCMEncodedFrameCallback callback(&sink, nullptr);
+  // Any non-zero bitrate needed to be set before the first frame.
+  callback.OnTargetBitrateChanged(500, 0);
   callback.OnEncodeStarted(timestamp1, 0);
   EXPECT_EQ(0u, sink.GetNumFramesDropped());
   image.capture_time_ms_ = timestamp1;