Add encode time to WebRTC telemetry tests, remove some noisy metrics.

tools/perf/metrics/webrtc_stats_unittest.py

 # 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.
 
 import unittest
 
 from telemetry.testing import simple_mock
 
 from metrics import webrtc_stats
 
@@ skipping 111 matching lines @@
     page.url = simple_mock.MockObject()
     results = FakeResults(page)
     stats_metric.AddResults(tab, results)
     return results
 
   def testExtractsValuesAsTimeSeries(self):
     results = self._RunMetricOnJson(SAMPLE_JSON)
 
     self.assertTrue(results.received_values,
                     'Expected values for googDecodeMs and others, got none.')
-
-    # This also ensures we're clever enough to tell video packetsSent from audio
-    # packetsSent.
-    self.assertEqual(results.received_values[3].values,
-                     [4.0, 16.0])
-    self.assertEqual(results.received_values[5].values,
-                     [1.0, 8.0])
+    self.assertEqual(results.received_values[1].name,
+                     'peer_connection_0_audio_goog_rtt')
+    self.assertEqual(results.received_values[1].values,
+                     [20.0, 17.0])
+    self.assertEqual(results.received_values[7].name,
+                     'peer_connection_1_video_goog_rtt')
+    self.assertEqual(results.received_values[7].values,
+                     [100.0, 101.0])
 
   def testExtractsInterestingMetricsOnly(self):
     results = self._RunMetricOnJson(SAMPLE_JSON)
 
     self.assertTrue(len(results.received_values) > 0)
     self.assertIn('peer_connection_0', results.received_values[0].name,
                   'The result should be a ListOfScalarValues instance with '
                   'a name <peer connection id>_<statistic>.')
     all_names = [value.name for value in results.received_values]
-    self.assertIn('peer_connection_0_video_packets_sent', all_names)
-    self.assertNotIn('peer_connection_1_video_packets_sent', all_names,
-                     'Peer connection 1 does not have a video packets sent in '
+    self.assertIn('peer_connection_0_audio_goog_rtt', all_names)
+    self.assertNotIn('peer_connection_1_audio_goog_rtt', all_names,
+                     'Peer connection 1 does not have a goog-rtt in '
                      'the JSON above, unlike peer connection 0 which does.')
     self.assertIn('peer_connection_0_video_goog_rtt', all_names)
     self.assertIn('peer_connection_1_video_goog_rtt', all_names)
     # The audio_audio is intentional since the code distinguishes audio reports
     # from video reports (even though audio_input_level is quite obvious).
     self.assertNotIn('peer_connection_0_audio_audio_input_level', all_names,
                      'Input level is in the JSON for both connections but '
                      'should not be reported since it is not interesting.')
     self.assertNotIn('peer_connection_1_audio_audio_input_level', all_names)
 
   def testReturnsIfJsonIsEmpty(self):
     results = self._RunMetricOnJson('[]')
     self.assertFalse(results.received_values)