From 7232ad78b2c72934b5a55aad050a80a1f1bb2bde Mon Sep 17 00:00:00 2001 From: "pwestin@webrtc.org" Date: Thu, 3 Nov 2011 00:36:32 +0000 Subject: [PATCH] reverted back the sanity and changed the test Review URL: http://webrtc-codereview.appspot.com/254006 git-svn-id: http://webrtc.googlecode.com/svn/trunk@877 4adac7df-926f-26a2-2b94-8c16560cd09d --- src/modules/rtp_rtcp/source/rtp_utility.cc | 2 +- src/modules/rtp_rtcp/source/rtp_utility_test.cc | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/modules/rtp_rtcp/source/rtp_utility.cc b/src/modules/rtp_rtcp/source/rtp_utility.cc index 5c7e18a708..08be8a07d7 100644 --- a/src/modules/rtp_rtcp/source/rtp_utility.cc +++ b/src/modules/rtp_rtcp/source/rtp_utility.cc @@ -889,7 +889,7 @@ int ModuleRTPUtility::RTPPayloadParser::ParseVP8FrameSize( // Included in payload header for I-frames. return 0; } - if (dataLength < 7) + if (dataLength < 10) { // For an I-frame we should always have the uncompressed VP8 header // in the beginning of the partition. diff --git a/src/modules/rtp_rtcp/source/rtp_utility_test.cc b/src/modules/rtp_rtcp/source/rtp_utility_test.cc index 07b7c7cc2a..1031eb109c 100644 --- a/src/modules/rtp_rtcp/source/rtp_utility_test.cc +++ b/src/modules/rtp_rtcp/source/rtp_utility_test.cc @@ -132,12 +132,12 @@ TEST(ParseVP8Test, PictureID) { } TEST(ParseVP8Test, Tl0PicIdx) { - WebRtc_UWord8 payload[10] = {0}; + WebRtc_UWord8 payload[13] = {0}; payload[0] = 0x90; payload[1] = 0x40; payload[2] = 17; - RTPPayloadParser rtpPayloadParser(webrtc::kRtpVp8Video, payload, 10, 0); + RTPPayloadParser rtpPayloadParser(webrtc::kRtpVp8Video, payload, 13, 0); RTPPayload parsedPacket; ASSERT_TRUE(rtpPayloadParser.Parse(parsedPacket)); @@ -151,7 +151,7 @@ TEST(ParseVP8Test, Tl0PicIdx) { EXPECT_EQ(17, parsedPacket.info.VP8.tl0PicIdx); EXPECT_EQ(payload + 3, parsedPacket.info.VP8.data); - EXPECT_EQ(10 - 3, parsedPacket.info.VP8.dataLength); + EXPECT_EQ(13 - 3, parsedPacket.info.VP8.dataLength); } TEST(ParseVP8Test, TID) {