Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4486)

Unified Diff: chrome/browser/media/webrtc/webrtc_browsertest_perf.cc

Issue 2568133006: Delete flaky metrics from the browser perf tests. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/media/webrtc/webrtc_browsertest_perf.cc
diff --git a/chrome/browser/media/webrtc/webrtc_browsertest_perf.cc b/chrome/browser/media/webrtc/webrtc_browsertest_perf.cc
index 3a37ad321363940c91db963e018fbfa36a924e26..3d6b580be3e78b5eb8c35d131c773a9a1acf958c 100644
--- a/chrome/browser/media/webrtc/webrtc_browsertest_perf.cc
+++ b/chrome/browser/media/webrtc/webrtc_browsertest_perf.cc
@@ -31,9 +31,6 @@ static void MaybePrintResultsForAudioReceive(
return;
}
- EXPECT_TRUE(pc_dict.GetString(Statistic("bytesReceived", ssrc), &value));
- perf_test::PrintResult(
- "audio_bytes", modifier, "bytes_recv", value, "bytes", false);
EXPECT_TRUE(pc_dict.GetString(Statistic("packetsLost", ssrc), &value));
perf_test::PrintResult(
"audio_misc", modifier, "packets_lost", value, "frames", false);
@@ -64,9 +61,6 @@ static void MaybePrintResultsForAudioSend(
return;
}
- EXPECT_TRUE(pc_dict.GetString(Statistic("bytesSent", ssrc), &value));
- perf_test::PrintResult(
- "audio_bytes", modifier, "bytes_sent", value, "bytes", false);
EXPECT_TRUE(pc_dict.GetString(Statistic("googJitterReceived", ssrc), &value));
perf_test::PrintResult(
"audio_tx", modifier, "goog_jitter_recv", value, "ms", false);
@@ -97,10 +91,6 @@ static void MaybePrintResultsForVideoSend(
perf_test::PrintResult(
"video_fps", modifier, "goog_frame_rate_input", value, "fps", false);
- EXPECT_TRUE(pc_dict.GetString(Statistic("bytesSent", ssrc), &value));
- perf_test::PrintResult(
- "video_total_bytes", modifier, "bytes_sent", value, "bytes", false);
-
EXPECT_TRUE(pc_dict.GetString(Statistic("googFirsReceived", ssrc), &value));
perf_test::PrintResult(
"video_misc", modifier, "goog_firs_recv", value, "", false);
@@ -148,10 +138,6 @@ static void MaybePrintResultsForVideoReceive(
perf_test::PrintResult("video_misc", modifier, "packets_lost", value,
"frames", false);
- EXPECT_TRUE(pc_dict.GetString(Statistic("bytesReceived", ssrc), &value));
- perf_test::PrintResult(
- "video_total_bytes", modifier, "bytes_recv", value, "bytes", false);
-
EXPECT_TRUE(
pc_dict.GetString(Statistic("googFrameWidthReceived", ssrc), &value));
perf_test::PrintResult("video_resolution", modifier, "goog_frame_width_recv",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698