generic-library/webrtc
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

reverted back the sanity and changed the test

Browse Source 
Review URL: http://webrtc-codereview.appspot.com/254006

git-svn-id: http://webrtc.googlecode.com/svn/trunk@877 4adac7df-926f-26a2-2b94-8c16560cd09d
This commit is contained in:
pwestin@webrtc.org 2011-11-03 00:36:32 +00:00
parent cfc1070586
commit 7232ad78b2
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/modules/rtp_rtcp/source/rtp_utility.cc
View File

@ -889,7 +889,7 @@ int ModuleRTPUtility::RTPPayloadParser::ParseVP8FrameSize(
// Included in payload header for I-frames. // Included in payload header for I-frames.
return 0; return 0;
} }
if (dataLength < 7) if (dataLength < 10)
{ {
// For an I-frame we should always have the uncompressed VP8 header // For an I-frame we should always have the uncompressed VP8 header
// in the beginning of the partition. // in the beginning of the partition.

6
src/modules/rtp_rtcp/source/rtp_utility_test.cc
View File

@ -132,12 +132,12 @@ TEST(ParseVP8Test, PictureID) {
} }
TEST(ParseVP8Test, Tl0PicIdx) { TEST(ParseVP8Test, Tl0PicIdx) {
WebRtc_UWord8 payload[10] = {0}; WebRtc_UWord8 payload[13] = {0};
payload[0] = 0x90; payload[0] = 0x90;
payload[1] = 0x40; payload[1] = 0x40;
payload[2] = 17; payload[2] = 17;
RTPPayloadParser rtpPayloadParser(webrtc::kRtpVp8Video, payload, 10, 0); RTPPayloadParser rtpPayloadParser(webrtc::kRtpVp8Video, payload, 13, 0);
RTPPayload parsedPacket; RTPPayload parsedPacket;
ASSERT_TRUE(rtpPayloadParser.Parse(parsedPacket)); ASSERT_TRUE(rtpPayloadParser.Parse(parsedPacket));
@ -151,7 +151,7 @@ TEST(ParseVP8Test, Tl0PicIdx) {
EXPECT_EQ(17, parsedPacket.info.VP8.tl0PicIdx); EXPECT_EQ(17, parsedPacket.info.VP8.tl0PicIdx);
EXPECT_EQ(payload + 3, parsedPacket.info.VP8.data); 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) { TEST(ParseVP8Test, TID) {