commit 0379d8cfeac189a7fd815e23908b422b861592c7
author Artem Titov <titovartem@webrtc.org> Mon May 13 12:26:31 2019
committer Commit Bot <commit-bot@chromium.org> Mon May 13 13:07:01 2019
tree 099827cd0a3949ed086a02e7d50e135e9ce482e3
parent 157b7814b97ffa4df36afd817dc2b363affad17a

Change a way, how receive stream is determined in DefaultAudioQualityAnalyzer.

Bug: webrtc:10138
Change-Id: I8955c30f0a5d98abeca029323396e469a2fb243b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/136683
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27927}

test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc

diff --git a/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc b/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc
index 0792069..07f3d63 100644
--- a/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc
+++ b/test/pc/e2e/analyzer/audio/default_audio_quality_analyzer.cc
@@ -45,15 +45,10 @@
     if (strcmp(media_type->static_string_val(), kStatsAudioMediaType) != 0) {
       continue;
     }
-    const webrtc::StatsReport::Value* bytes_received = stats_report->FindValue(
-        StatsReport::StatsValueName::kStatsValueNameBytesReceived);
-    if (bytes_received == nullptr || bytes_received->int64_val() == 0) {
-      // Discarding stats in the following situations:
-      // - When bytes_received is not present, because NetEq stats are only
-      //   available in recv-side SSRC.
-      // - When bytes_received is present but its value is 0. This means
-      //   that media is not yet flowing so there is no need to keep this
-      //   stats report into account (since all its fields would be 0).
+    if (stats_report->FindValue(
+            webrtc::StatsReport::kStatsValueNameBytesSent)) {
+      // If kStatsValueNameBytesSent is present, it means it's a send stream,
+      // but we need audio metrics for receive stream, so skip it.
       continue;
     }