Chromium Code Reviews| Index: webrtc/pc/rtcstats_integrationtest.cc |
| diff --git a/webrtc/pc/rtcstats_integrationtest.cc b/webrtc/pc/rtcstats_integrationtest.cc |
| index 3337c4ee8b4944e41646780acb0c427bd1a61544..69b6ca62b01401cbb85f57562a7f20f0b350e225 100644 |
| --- a/webrtc/pc/rtcstats_integrationtest.cc |
| +++ b/webrtc/pc/rtcstats_integrationtest.cc |
| @@ -475,6 +475,8 @@ class RTCStatsReportVerifier { |
| verifier.TestMemberIsUndefined(media_stream_track.echo_return_loss); |
| verifier.TestMemberIsUndefined( |
| media_stream_track.echo_return_loss_enhancement); |
| + verifier.TestMemberIsUndefined(media_stream_track.total_audio_energy); |
| + verifier.TestMemberIsUndefined(media_stream_track.total_samples_duration); |
| } else { |
| RTC_DCHECK_EQ(*media_stream_track.kind, |
| RTCMediaStreamTrackKind::kAudio); |
| @@ -491,6 +493,12 @@ class RTCStatsReportVerifier { |
| verifier.TestMemberIsUndefined(media_stream_track.full_frames_lost); |
| // Audio-only members |
| verifier.TestMemberIsNonNegative<double>(media_stream_track.audio_level); |
| + // Remote audio tracks are expected to have audio energy/duration values |
|
Taylor Brandstetter
2017/07/06 17:57:56
Since you got this to work for both remote and loc
Zach Stein
2017/07/06 18:07:35
Done.
|
| + // that can be used to compute audio levels between getStats calls. |
| + verifier.TestMemberIsNonNegative<double>( |
| + media_stream_track.total_audio_energy); |
| + verifier.TestMemberIsNonNegative<double>( |
| + media_stream_track.total_samples_duration); |
| // TODO(hbos): |echo_return_loss| and |echo_return_loss_enhancement| are |
| // flaky on msan bot (sometimes defined, sometimes undefined). Should the |
| // test run until available or is there a way to have it always be |