commit 375b346b30a42eafc8f399fa5e92cd7414e81112
author Niels Möller <nisse@webrtc.org> Thu Jan 10 14:35:56 2019
committer Commit Bot <commit-bot@chromium.org> Wed Jan 16 15:41:07 2019
tree c4809c86c78b77e429ba3c03221b3246f972888d
parent 309aafe3513af2c53439cb0ea105a39156547acb

Delete all logic related to VCMDecodeErrorMode

Bug: webrtc:8064
Change-Id: I345f342a314d88390fff8b305b121076b45a51e8
Reviewed-on: https://webrtc-review.googlesource.com/c/116692
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Erik Språng <sprang@webrtc.org>
Commit-Queue: Niels Moller <nisse@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26283}

modules/video_coding/frame_buffer.cc

diff --git a/modules/video_coding/frame_buffer.cc b/modules/video_coding/frame_buffer.cc
index 908a2a2..37709c9 100644
--- a/modules/video_coding/frame_buffer.cc
+++ b/modules/video_coding/frame_buffer.cc
@@ -80,7 +80,6 @@
 VCMFrameBufferEnum VCMFrameBuffer::InsertPacket(
     const VCMPacket& packet,
     int64_t timeInMs,
-    VCMDecodeErrorMode decode_error_mode,
     const FrameData& frame_data) {
   TRACE_EVENT0("webrtc", "VCMFrameBuffer::InsertPacket");
   assert(!(NULL == packet.dataPtr && packet.sizeBytes > 0));
@@ -129,8 +128,7 @@
   if (packet.sizeBytes > 0)
     CopyCodecSpecific(&packet.video_header);
 
-  int retVal =
-      _sessionInfo.InsertPacket(packet, _buffer, decode_error_mode, frame_data);
+  int retVal = _sessionInfo.InsertPacket(packet, _buffer, frame_data);
   if (retVal == -1) {
     return kSizeError;
   } else if (retVal == -2) {
@@ -182,9 +180,6 @@
   if (_sessionInfo.complete()) {
     SetState(kStateComplete);
     return kCompleteSession;
-  } else if (_sessionInfo.decodable()) {
-    SetState(kStateDecodable);
-    return kDecodableSession;
   }
   return kIncomplete;
 }
@@ -240,8 +235,7 @@
       break;
 
     case kStateComplete:
-      assert(_state == kStateEmpty || _state == kStateIncomplete ||
-             _state == kStateDecodable);
+      assert(_state == kStateEmpty || _state == kStateIncomplete);
 
       break;
 
@@ -249,10 +243,6 @@
       // Should only be set to empty through Reset().
       assert(false);
       break;
-
-    case kStateDecodable:
-      assert(_state == kStateEmpty || _state == kStateIncomplete);
-      break;
   }
   _state = state;
 }