diff --git a/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc b/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc index 73f774d635..3c1b5f8fd2 100644 --- a/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc +++ b/webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.cc @@ -604,7 +604,7 @@ TEST_F(VideoProcessorIntegrationTest, Process0PercentPacketLossVP9) { false, true, false); // Metrics for expected quality. QualityMetrics quality_metrics; - SetQualityMetrics(&quality_metrics, 37.5, 36.0, 0.94, 0.93); + SetQualityMetrics(&quality_metrics, 37.0, 36.0, 0.93, 0.92); // Metrics for rate control. RateControlMetrics rc_metrics[1]; SetRateControlMetrics(rc_metrics, 0, 0, 40, 20, 10, 15, 0); @@ -657,7 +657,7 @@ TEST_F(VideoProcessorIntegrationTest, ProcessNoLossChangeBitRateVP9) { false, true, false); // Metrics for expected quality. QualityMetrics quality_metrics; - SetQualityMetrics(&quality_metrics, 36.0, 32.0, 0.90, 0.85); + SetQualityMetrics(&quality_metrics, 36.0, 31.8, 0.90, 0.85); // Metrics for rate control. RateControlMetrics rc_metrics[3]; SetRateControlMetrics(rc_metrics, 0, 0, 30, 20, 20, 20, 0); @@ -692,11 +692,11 @@ TEST_F(VideoProcessorIntegrationTest, false, true, false); // Metrics for expected quality. QualityMetrics quality_metrics; - SetQualityMetrics(&quality_metrics, 30.0, 18.0, 0.80, 0.40); + SetQualityMetrics(&quality_metrics, 29.0, 17.0, 0.80, 0.40); // Metrics for rate control. RateControlMetrics rc_metrics[3]; - SetRateControlMetrics(rc_metrics, 0, 35, 55, 70, 15, 40, 0); - SetRateControlMetrics(rc_metrics, 1, 15, 0, 50, 10, 30, 0); + SetRateControlMetrics(rc_metrics, 0, 50, 60, 100, 15, 45, 0); + SetRateControlMetrics(rc_metrics, 1, 30, 0, 65, 10, 35, 0); SetRateControlMetrics(rc_metrics, 2, 5, 0, 38, 10, 30, 0); ProcessFramesAndVerify(quality_metrics, rate_profile, diff --git a/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc b/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc index 33f11a373a..734e73d27f 100644 --- a/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc +++ b/webrtc/modules/video_coding/codecs/vp9/vp9_impl.cc @@ -189,11 +189,8 @@ int VP9EncoderImpl::InitAndSetControlSettings(const VideoCodec* inst) { return WEBRTC_VIDEO_CODEC_UNINITIALIZED; } // Only positive speeds, currently: 0 - 7. - // O means slowest/best quality, 7 means fastest/lowest quality. - // TODO(marpan): Speeds 5-7 are speed settings for real-time mode, on desktop. - // Currently set to 5, update to 6 (for faster encoding) after some subjective - // quality tests. - cpu_speed_ = 5; + // O means slowest/best quality, 7 means fastest/lower quality. + cpu_speed_ = 6; // Note: some of these codec controls still use "VP8" in the control name. // TODO(marpan): Update this in the next/future libvpx version. vpx_codec_control(encoder_, VP8E_SET_CPUUSED, cpu_speed_); diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc index 80d8e19a78..2b3c00f5f5 100644 --- a/webrtc/video/end_to_end_tests.cc +++ b/webrtc/video/end_to_end_tests.cc @@ -225,8 +225,7 @@ TEST_F(EndToEndTest, TransmitsFirstFrame) { DestroyStreams(); } -// TODO(marpan): Re-enable this test on the next libvpx roll. -TEST_F(EndToEndTest, DISABLED_SendsAndReceivesVP9) { +TEST_F(EndToEndTest, SendsAndReceivesVP9) { class VP9Observer : public test::EndToEndTest, public VideoRenderer { public: VP9Observer()