Only update codec type histogram if lifetime is long enough (10 sec).

webrtc/video/end_to_end_tests.cc

Index: webrtc/video/end_to_end_tests.cc
diff --git a/webrtc/video/end_to_end_tests.cc b/webrtc/video/end_to_end_tests.cc
index 8e3105f7411d9af37012cb7ea6635ef7edf4b79a..8e01bb755ca7fe7a2841636a5e3cd84a2bb58b6f 100644
--- a/webrtc/video/end_to_end_tests.cc
+++ b/webrtc/video/end_to_end_tests.cc
@@ -2130,12 +2130,17 @@ void EndToEndTest::VerifyHistogramStats(bool use_rtx,
       screenshare ? "WebRTC.Video.Screenshare." : "WebRTC.Video.";
 
   // Verify that stats have been updated once.
+  EXPECT_EQ(2, metrics::NumSamples("WebRTC.Call.LifetimeInSeconds"));

[pbos-webrtc, 2016/07/10 17:43:03]

This sounds very flaky to me, deleting the call etc. might not be instant. Maybe
EXPECT_GE?

[åsapersson, 2016/07/25 07:54:04]

Checks that the metric is reported twice, once for the sender_call and once for
the receiver_call.

   EXPECT_EQ(1, metrics::NumSamples("WebRTC.Call.VideoBitrateReceivedInKbps"));
   EXPECT_EQ(1, metrics::NumSamples("WebRTC.Call.RtcpBitrateReceivedInBps"));
   EXPECT_EQ(1, metrics::NumSamples("WebRTC.Call.BitrateReceivedInKbps"));
   EXPECT_EQ(1, metrics::NumSamples("WebRTC.Call.EstimatedSendBitrateInKbps"));
   EXPECT_EQ(1, metrics::NumSamples("WebRTC.Call.PacerBitrateInKbps"));
 
+  EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.SendStreamLifetimeInSeconds"));

[pbos-webrtc, 2016/07/10 17:43:03]

Same here?

[åsapersson, 2016/07/25 07:54:04]

Reported once for the send/receive stream.

+  EXPECT_EQ(1,
+            metrics::NumSamples("WebRTC.Video.ReceiveStreamLifetimeInSeconds"));
+
   EXPECT_EQ(1, metrics::NumSamples("WebRTC.Video.NackPacketsSentPerMinute"));
   EXPECT_EQ(1,
             metrics::NumSamples(video_prefix + "NackPacketsReceivedPerMinute"));