admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Vp9: Enable denoiser by default.

Browse Source 
BUG=webrtc:7412

Review-Url: https://codereview.webrtc.org/2765663002
Cr-Commit-Position: refs/heads/master@{#17395}
This commit is contained in:
jianj 2017-03-27 10:09:00 -07:00 committed by Commit bot
parent 11173f7ec5
commit a5e8aa6e2b
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
webrtc/media/engine/webrtcvideoengine2.cc
View File

@ -409,7 +409,7 @@ WebRtcVideoChannel2::WebRtcVideoSendStream::ConfigureVideoEncoderSettings(
vp9_settings.numberOfSpatialLayers = GetDefaultVp9SpatialLayers();
}
// VP9 denoising is disabled by default.
vp9_settings.denoisingOn = codec_default_denoising ? false : denoising;
vp9_settings.denoisingOn = codec_default_denoising ? true : denoising;
vp9_settings.frameDroppingOn = frame_dropping;
return new rtc::RefCountedObject<
webrtc::VideoEncoderConfig::Vp9EncoderSpecificSettings>(vp9_settings);

4
webrtc/media/engine/webrtcvideoengine2_unittest.cc
View File

@ -1971,8 +1971,8 @@ TEST_F(Vp9SettingsTest, VerifyVp9SpecificSettings) {
webrtc::VideoCodecVP9 vp9_settings;
ASSERT_TRUE(stream->GetVp9Settings(&vp9_settings)) << "No VP9 config set.";
EXPECT_FALSE(vp9_settings.denoisingOn)
<< "VP9 denoising should be off by default.";
EXPECT_TRUE(vp9_settings.denoisingOn)
<< "VP9 denoising should be on by default.";
stream = SetDenoisingOption(last_ssrc_, &capturer, false);