RTCMediaStreamTrackStats.framesDecoded collected.

webrtc/api/rtcstatscollector_unittest.cc

Index: webrtc/api/rtcstatscollector_unittest.cc
diff --git a/webrtc/api/rtcstatscollector_unittest.cc b/webrtc/api/rtcstatscollector_unittest.cc
index ac339193b998b4bc4313cb30431f1bd0275dcaf3..d0bbcd197e502b378cf21ca726f23a1d8365dee1 100644
--- a/webrtc/api/rtcstatscollector_unittest.cc
+++ b/webrtc/api/rtcstatscollector_unittest.cc
@@ -1533,6 +1533,7 @@ TEST_F(RTCStatsCollectorTest,
   video_receiver_info_ssrc3.local_stats[0].ssrc = 3;
   video_receiver_info_ssrc3.frame_width = 6789;
   video_receiver_info_ssrc3.frame_height = 9876;
+  video_receiver_info_ssrc3.frames_decoded = 1234;
 
   // Remote audio track with undefined (default) values
   rtc::scoped_refptr<MediaStreamTrackInterface> remote_video_track_ssrc4 =
@@ -1546,6 +1547,7 @@ TEST_F(RTCStatsCollectorTest,
   video_receiver_info_ssrc4.local_stats[0].ssrc = 4;
   video_receiver_info_ssrc4.frame_width = 0;
   video_receiver_info_ssrc4.frame_height = 0;
+  video_receiver_info_ssrc4.frames_decoded = 0;
 
   test_->CreateMockRtpSendersReceiversAndChannels(
       {}, {},
@@ -1618,6 +1620,7 @@ TEST_F(RTCStatsCollectorTest,
   expected_remote_video_track_ssrc3.detached = false;
   expected_remote_video_track_ssrc3.frame_width = 6789;
   expected_remote_video_track_ssrc3.frame_height = 9876;
+  expected_remote_video_track_ssrc3.frames_decoded = 1234;
   ASSERT_TRUE(report->Get(expected_remote_video_track_ssrc3.id()));
   EXPECT_EQ(expected_remote_video_track_ssrc3,
             report->Get(expected_remote_video_track_ssrc3.id())->cast_to<
@@ -1631,6 +1634,7 @@ TEST_F(RTCStatsCollectorTest,
   expected_remote_video_track_ssrc4.remote_source = true;
   expected_remote_video_track_ssrc4.ended = false;
   expected_remote_video_track_ssrc4.detached = false;
+  expected_remote_video_track_ssrc4.frames_decoded = 0;

[hta-webrtc, 2017/01/20 09:41:36]

If we expect in the future that two SSRCs may be associated with an incoming
track, we should maybe add a test for it now - see that it behaves as expected
(I don't care if it's a DCHECK, a sum or a number that is less than the max - as
long as it's the behavior you expect).

If we can't do it now, just leave it.

[hbos, 2017/01/20 14:45:48]

Can't do it now, the assumption about only having one receiver info per remote
track is DCHECKed, testing that would crash.

Unless you mean multiple SSRCs for the same receiver, that is possible, but only
the first one is used as an ID, the rest don't matter. (I don't fully understand
what multiple SSRCs in the same "group" like that means but it doesn't affect
this AFAIK.)

   // Should be undefined: |expected_remote_video_track_ssrc4.frame_width| and
   // |expected_remote_video_track_ssrc4.frame_height|.
   ASSERT_TRUE(report->Get(expected_remote_video_track_ssrc4.id()));