Correcting value incorrectly logged as video_send.

chrome/browser/media/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_browsertest_perf.h"
 
 #include "base/strings/stringprintf.h"
 #include "base/values.h"
 #include "chrome/test/base/in_process_browser_test.h"
 #include "testing/perf/perf_test.h"
@@ skipping 65 matching lines @@
   perf_test::PrintResult("video_recv", "", "goog_nacks_recv", value, "", false);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googFrameWidthSent", ssrc), &value));
   perf_test::PrintResult("video_recv", "", "goog_frame_width_sent", value,
                          "pixels", false);
   EXPECT_TRUE(
       pc_dict.GetString(Statistic("googFrameHeightSent", ssrc), &value));
   perf_test::PrintResult("video_recv", "", "goog_frame_height_sent", value,
                          "pixels", false);
   EXPECT_TRUE(pc_dict.GetString(
       Statistic("googCaptureJitterMs", ssrc), &value));
-  perf_test::PrintResult("video_send", "", "goog_capture_jitter_ms", value,
+  perf_test::PrintResult("video_recv", "", "goog_capture_jitter_ms", value,

[tommi (sloooow) - chröme, 2014/03/28 09:36:25]

can we have constants in the functions so that we don't have to repeat the
constant.

[phoglund_chromium, 2014/03/28 10:05:18]

Yeah, that's a good idea in this case (would have prevented this particular
bug).

                          "ms", false);
   EXPECT_TRUE(pc_dict.GetString(
       Statistic("googCaptureQueueDelayMsPerS", ssrc), &value));
-  perf_test::PrintResult("video_send", "", "goog_capture_queue_delay_ms_per_s",
+  perf_test::PrintResult("video_recv", "", "goog_capture_queue_delay_ms_per_s",
                          value, "ms/s", false);
   EXPECT_TRUE(pc_dict.GetString(
       Statistic("googEncodeUsagePercent", ssrc), &value));
-  perf_test::PrintResult("video_send", "", "goog_encode_usage_percent",
+  perf_test::PrintResult("video_recv", "", "goog_encode_usage_percent",
                          value, "%", false);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googAvgEncodeMs", ssrc), &value));
-  perf_test::PrintResult("video_send", "", "goog_avg_encode_ms", value, "ms",
+  perf_test::PrintResult("video_recv", "", "goog_avg_encode_ms", value, "ms",
                          false);
   EXPECT_TRUE(pc_dict.GetString(Statistic("googRtt", ssrc), &value));
-  perf_test::PrintResult("video_send", "", "goog_rtt", value, "ms", false);
+  perf_test::PrintResult("video_recv", "", "goog_rtt", value, "ms", false);
   return true;
 }
 
 static bool MaybePrintResultsForVideoSend(
     const std::string& ssrc, const base::DictionaryValue& pc_dict) {
   std::string value;
   if (!pc_dict.GetString(Statistic("googFrameRateOutput", ssrc), &value)) {
     // Not a video receive stream.
     return false;
   }
@@ skipping 104 matching lines @@
         MaybePrintResultsForAudioSend(ssrc, pc_dict) ||
         MaybePrintResultsForVideoReceive(ssrc, pc_dict) ||
         MaybePrintResultsForVideoSend(ssrc, pc_dict);
     ASSERT_TRUE(did_recognize_stream_type) << "Failed to figure out which "
                                               "kind of stream SSRC " << ssrc
                                            << " is. ";
   }
 }
 
 }  // namespace test