Bug 1497974 - Remove local changes to jitter_buffer.cc; r=pehrsons
authorDan Minor <dminor@mozilla.com>
Fri, 07 Dec 2018 20:18:57 +0000
changeset 508856 2fb1dd0d8344ef24e2e0fca94725bbfaf59aa257
parent 508855 71253f35ac3cf2fa6f1c7b613443c2d494809c6a
child 508857 67a5e68acfc982df3b1eb2f2c5112f055dc444e6
push id1905
push userffxbld-merge
push dateMon, 21 Jan 2019 12:33:13 +0000
treeherdermozilla-release@c2fca1944d8c [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerspehrsons
bugs1497974
milestone65.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1497974 - Remove local changes to jitter_buffer.cc; r=pehrsons These modifications are made to code which we do not use and so can be removed. Differential Revision: https://phabricator.services.mozilla.com/D13989
media/webrtc/trunk/webrtc/modules/video_coding/jitter_buffer.cc
--- a/media/webrtc/trunk/webrtc/modules/video_coding/jitter_buffer.cc
+++ b/media/webrtc/trunk/webrtc/modules/video_coding/jitter_buffer.cc
@@ -568,22 +568,16 @@ VCMEncodedFrame* VCMJitterBuffer::Extrac
 
   // We have a frame - update the last decoded state and nack list.
   last_decoded_state_.SetState(frame);
   DropPacketsFromNackList(last_decoded_state_.sequence_num());
 
   if ((*frame).IsSessionComplete())
     UpdateAveragePacketsPerFrame(frame->NumPackets());
 
-  if (frame->Length() == 0) {
-    // Normally only if MakeDecodable() on an incomplete frame threw it all away
-    ReleaseFrame(frame);
-    return NULL;
-  }
-
   return frame;
 }
 
 // Release frame when done with decoding. Should never be used to release
 // frames from within the jitter buffer.
 void VCMJitterBuffer::ReleaseFrame(VCMEncodedFrame* frame) {
   RTC_CHECK(frame != nullptr);
   rtc::CritScope cs(&crit_sect_);