diff --git a/media/base/mediachannel.h b/media/base/mediachannel.h index 108ae26864..3ab333ec7c 100644 --- a/media/base/mediachannel.h +++ b/media/base/mediachannel.h @@ -428,14 +428,6 @@ struct VoiceSenderInfo : public MediaSenderInfo { // https://w3c.github.io/webrtc-stats/#dom-rtcmediastreamtrackstats-totalaudioenergy double total_input_energy = 0.0; double total_input_duration = 0.0; - // TODO(bugs.webrtc.org/8572): Remove APM stats from this struct, since they - // are no longer needed now that we have apm_statistics. - int echo_delay_median_ms = 0; - int echo_delay_std_ms = 0; - int echo_return_loss = 0; - int echo_return_loss_enhancement = 0; - float residual_echo_likelihood = 0.0f; - float residual_echo_likelihood_recent_max = 0.0f; bool typing_noise_detected = false; webrtc::ANAStats ana_statistics; webrtc::AudioProcessingStats apm_statistics; diff --git a/pc/statscollector_unittest.cc b/pc/statscollector_unittest.cc index 4a4be413d9..53eb2eaa5e 100644 --- a/pc/statscollector_unittest.cc +++ b/pc/statscollector_unittest.cc @@ -479,10 +479,10 @@ void InitVoiceSenderInfo(cricket::VoiceSenderInfo* voice_sender_info) { voice_sender_info->packets_lost = 105; voice_sender_info->ext_seqnum = 106; voice_sender_info->audio_level = 107; - voice_sender_info->echo_return_loss = 108; - voice_sender_info->echo_return_loss_enhancement = 109; - voice_sender_info->echo_delay_median_ms = 110; - voice_sender_info->echo_delay_std_ms = 111; + voice_sender_info->apm_statistics.echo_return_loss = 108; + voice_sender_info->apm_statistics.echo_return_loss_enhancement = 109; + voice_sender_info->apm_statistics.delay_median_ms = 110; + voice_sender_info->apm_statistics.delay_standard_deviation_ms = 111; voice_sender_info->typing_noise_detected = false; voice_sender_info->ana_statistics.bitrate_action_counter = 112; voice_sender_info->ana_statistics.channel_action_counter = 113;