Bug 1002402: typo fix for adjusting SPS/PPS timestamps r=jesup
authorJohn Lin <jolin@mozilla.com>
Wed, 30 Apr 2014 01:20:41 -0400
changeset 181345 edd20e5bac2cca26da40ffd2a625d3c5602b0c35
parent 181344 10e37b92e195e1fa441c1ae76d1a9c976c66be00
child 181346 7f1eda6d68cbce5519a69b545cb1163373e863d9
push id272
push userpvanderbeken@mozilla.com
push dateMon, 05 May 2014 16:31:18 +0000
reviewersjesup
bugs1002402
milestone32.0a1
Bug 1002402: typo fix for adjusting SPS/PPS timestamps r=jesup
media/webrtc/signaling/src/media-conduit/WebrtcOMXH264VideoCodec.cpp
--- a/media/webrtc/signaling/src/media-conduit/WebrtcOMXH264VideoCodec.cpp
+++ b/media/webrtc/signaling/src/media-conduit/WebrtcOMXH264VideoCodec.cpp
@@ -615,16 +615,17 @@ private:
       nalu._length = nalSize;
       // [Workaround] Jitter buffer code in WebRTC.org cannot correctly deliver
       // SPS/PPS/I-frame NALUs that share same timestamp. Change timestamp value
       // for SPS/PPS to avoid it.
       // TODO: remove when bug 985254 lands. The patch there has same workaround.
       switch (nalStart[0] & 0x1f) {
         case 7:
           nalu._timeStamp -= 100;
+          break;
         case 8:
           nalu._timeStamp -= 50;
           break;
         default:
           break;
       }
       mCallback->Encoded(nalu, nullptr, nullptr);
     }