Delete flaky metrics from the browser perf tests.

chrome/browser/media/webrtc/webrtc_browsertest_perf.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/media/webrtc/webrtc_browsertest_perf.h"
 
 #include <stddef.h>
 
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
@@ skipping 13 matching lines @@
 
 static void MaybePrintResultsForAudioReceive(
     const std::string& ssrc, const base::DictionaryValue& pc_dict,
     const std::string& modifier) {
   std::string value;
   if (!pc_dict.GetString(Statistic("audioOutputLevel", ssrc), &value)) {
     // Not an audio receive stream.
     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);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googJitterReceived", ssrc), &value));
   perf_test::PrintResult(
       "audio_rx", modifier, "goog_jitter_recv", value, "ms", false);
 
   EXPECT_TRUE(pc_dict.GetString(Statistic("googExpandRate", ssrc), &value));
   perf_test::PrintResult(
       "audio_rates", modifier, "goog_expand_rate", value, "%", false);
@@ skipping 10 matching lines @@
 
 static void MaybePrintResultsForAudioSend(
     const std::string& ssrc, const base::DictionaryValue& pc_dict,
     const std::string& modifier) {
   std::string value;
   if (!pc_dict.GetString(Statistic("audioInputLevel", ssrc), &value)) {
     // Not an audio send stream.
     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);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googRtt", ssrc), &value));
   perf_test::PrintResult(
       "audio_tx", modifier, "goog_rtt", value, "ms", false);
   EXPECT_TRUE(
       pc_dict.GetString(Statistic("packetsSentPerSecond", ssrc), &value));
   perf_test::PrintResult("audio_tx", modifier, "packets_sent_per_second", value,
                          "packets", false);
@@ skipping 10 matching lines @@
 
   // Graph these by unit: the dashboard expects all stats in one graph to have
   // the same unit (e.g. ms, fps, etc). Most graphs, like video_fps, will also
   // be populated by the counterparts on the video receiving side.
   perf_test::PrintResult(
       "video_fps", modifier, "goog_frame_rate_sent", value, "fps", false);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googFrameRateInput", ssrc), &value));
   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);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googNacksReceived", ssrc), &value));
   perf_test::PrintResult(
       "video_misc", modifier, "goog_nacks_recv", value, "", false);
 
   EXPECT_TRUE(pc_dict.GetString(Statistic("googFrameWidthSent", ssrc), &value));
   perf_test::PrintResult("video_resolution", modifier, "goog_frame_width_sent",
                          value, "pixels", false);
@@ skipping 27 matching lines @@
       "video_fps", modifier, "goog_frame_rate_recv", value, "fps", false);
   EXPECT_TRUE(
       pc_dict.GetString(Statistic("googFrameRateOutput", ssrc), &value));
   perf_test::PrintResult(
       "video_fps", modifier, "goog_frame_rate_output", value, "fps", false);
 
   EXPECT_TRUE(pc_dict.GetString(Statistic("packetsLost", ssrc), &value));
   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",
                          value, "pixels", false);
   EXPECT_TRUE(
       pc_dict.GetString(Statistic("googFrameHeightReceived", ssrc), &value));
   perf_test::PrintResult("video_resolution", modifier, "goog_frame_height_recv",
                          value, "pixels", false);
 
   EXPECT_TRUE(pc_dict.GetString(Statistic("googCurrentDelayMs", ssrc), &value));
@@ skipping 107 matching lines @@
   std::set<std::string>::const_iterator ssrc_iterator =
       ssrc_identifiers.begin();
   for (; ssrc_iterator != ssrc_identifiers.end(); ++ssrc_iterator) {
     const std::string& ssrc = *ssrc_iterator;
     MaybePrintResultsForAudioReceive(ssrc, pc_dict, modifier);
     MaybePrintResultsForVideoReceive(ssrc, pc_dict, video_modifier);
   }
 }
 
 }  // namespace test