Expose RtpCodecParameters to VideoMediaInfo stats.

webrtc/media/base/videoengine_unittest.h

 /*
  *  Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 405 matching lines @@
 
     cricket::VideoMediaInfo info;
     EXPECT_TRUE(channel_->GetStats(&info));
 
     ASSERT_EQ(1U, info.senders.size());
     // TODO(whyuan): bytes_sent and bytes_rcvd are different. Are both payload?
     // For webrtc, bytes_sent does not include the RTP header length.
     EXPECT_GT(info.senders[0].bytes_sent, 0);
     EXPECT_EQ(NumRtpPackets(), info.senders[0].packets_sent);
     EXPECT_EQ(0.0, info.senders[0].fraction_lost);
+    EXPECT_TRUE(info.senders[0].codec_payload_type);
+    EXPECT_EQ(DefaultCodec().id, *info.senders[0].codec_payload_type);

[Taylor Brandstetter, 2016/11/08 23:55:44]

nit: This will crash if codec_payload_type if null (I think), so may want to
change the above EXPECT_TRUE to an ASSERT_TRUE.

[hbos, 2016/11/11 11:20:05]

Done. Here and for receivers.

     EXPECT_EQ(0, info.senders[0].firs_rcvd);
     EXPECT_EQ(0, info.senders[0].plis_rcvd);
     EXPECT_EQ(0, info.senders[0].nacks_rcvd);
     EXPECT_EQ(kVideoWidth, info.senders[0].send_frame_width);
     EXPECT_EQ(kVideoHeight, info.senders[0].send_frame_height);
     EXPECT_GT(info.senders[0].framerate_input, 0);
     EXPECT_GT(info.senders[0].framerate_sent, 0);
 
+    EXPECT_NE(info.send_codecs.end(), info.send_codecs.find(DefaultCodec().id));
+    EXPECT_EQ(DefaultCodec().ToCodecParameters(),
+              info.send_codecs[DefaultCodec().id]);
+
     ASSERT_EQ(1U, info.receivers.size());
     EXPECT_EQ(1U, info.senders[0].ssrcs().size());
     EXPECT_EQ(1U, info.receivers[0].ssrcs().size());
     EXPECT_EQ(info.senders[0].ssrcs()[0], info.receivers[0].ssrcs()[0]);
+    EXPECT_TRUE(info.receivers[0].codec_payload_type);
+    EXPECT_EQ(DefaultCodec().id, *info.receivers[0].codec_payload_type);
     EXPECT_EQ(NumRtpBytes(), info.receivers[0].bytes_rcvd);
     EXPECT_EQ(NumRtpPackets(), info.receivers[0].packets_rcvd);
     EXPECT_EQ(0.0, info.receivers[0].fraction_lost);
     EXPECT_EQ(0, info.receivers[0].packets_lost);
     // TODO(asapersson): Not set for webrtc. Handle missing stats.
     // EXPECT_EQ(0, info.receivers[0].packets_concealed);
     EXPECT_EQ(0, info.receivers[0].firs_sent);
     EXPECT_EQ(0, info.receivers[0].plis_sent);
     EXPECT_EQ(0, info.receivers[0].nacks_sent);
     EXPECT_EQ(kVideoWidth, info.receivers[0].frame_width);
     EXPECT_EQ(kVideoHeight, info.receivers[0].frame_height);
     EXPECT_GT(info.receivers[0].framerate_rcvd, 0);
     EXPECT_GT(info.receivers[0].framerate_decoded, 0);
     EXPECT_GT(info.receivers[0].framerate_output, 0);
+
+    EXPECT_NE(info.receive_codecs.end(),
+              info.receive_codecs.find(DefaultCodec().id));
+    EXPECT_EQ(DefaultCodec().ToCodecParameters(),
+              info.receive_codecs[DefaultCodec().id]);
   }
 
   cricket::VideoSenderInfo GetSenderStats(size_t i) {
     cricket::VideoMediaInfo info;
     EXPECT_TRUE(channel_->GetStats(&info));
     return info.senders[i];
   }
 
   cricket::VideoReceiverInfo GetReceiverStats(size_t i) {
     cricket::VideoMediaInfo info;
@@ skipping 465 matching lines @@
   std::unique_ptr<C> channel_;
   cricket::FakeNetworkInterface network_interface_;
   cricket::FakeVideoRenderer renderer_;
   cricket::VideoMediaChannel::Error media_error_;
 
   // Used by test cases where 2 streams are run on the same channel.
   cricket::FakeVideoRenderer renderer2_;
 };
 
 #endif  // WEBRTC_MEDIA_BASE_VIDEOENGINE_UNITTEST_H_  NOLINT