Make Call::OnRecoveredPacket parse RTP header and call OnRtpPacket.

webrtc/video/video_receive_stream_unittest.cc

Index: webrtc/video/video_receive_stream_unittest.cc
diff --git a/webrtc/video/video_receive_stream_unittest.cc b/webrtc/video/video_receive_stream_unittest.cc
index d536bf1dc61cf0d62cf7ec86317a7fbaa8c68cf9..a6a3ab28a01c39f4c17e56c473357d6aa423e91c 100644
--- a/webrtc/video/video_receive_stream_unittest.cc
+++ b/webrtc/video/video_receive_stream_unittest.cc
@@ -129,9 +129,9 @@ TEST_F(VideoReceiveStreamTest, CreateFrameFromH264FmtpSpropAndIdr) {
   EXPECT_CALL(mock_h264_video_decoder_, RegisterDecodeCompleteCallback(_));
   video_receive_stream_->Start();
   EXPECT_CALL(mock_h264_video_decoder_, Decode(_, false, _, _, _));
-  EXPECT_EQ(true,
-            video_receive_stream_->OnRecoveredPacket(rtppacket.data(),

[brandtr, 2017/05/10 14:27:29]

This looks weird to me. Why is OnRecoveredPacket called here in the first place?

[nisse-webrtc, 2017/05/11 07:36:15]

No idea. The test was added a few months ago, see
https://codereview.webrtc.org/2721653002

-                                                     rtppacket.size()));
+  RtpPacketReceived parsed_packet;
+  ASSERT_TRUE(parsed_packet.Parse(rtppacket.data(), rtppacket.size()));
+  video_receive_stream_->OnRtpPacket(parsed_packet);
   EXPECT_CALL(mock_h264_video_decoder_, Release());
   // Make sure the decoder thread had a chance to run.
   init_decode_event_.Wait(kDefaultTimeOutMs);